Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ungate dependency task modules #3511

Merged
merged 1 commit into from
Jul 29, 2024
Merged

Ungate dependency task modules #3511

merged 1 commit into from
Jul 29, 2024

Conversation

ss-es
Copy link
Contributor

@ss-es ss-es commented Jul 29, 2024

This PR:

In a separate PR, I was mildly surprised to see that just cargo check did not produce any errors for the dependency task modules when making a type change.

I don't believe there's any reason to gate the dependency task modules themselves; the main thing we want to gate is whether the dependency tasks are spawned in HotShot (which is already gated). This PR removes the gating on the modules.

This PR does not:

Dependency tasks will still not run unless the feature is enabled; this PR just prevents having to run cargo check twice (with and without the feature).

Key places to review:

@ss-es ss-es requested a review from bfish713 as a code owner July 29, 2024 17:38
@ss-es ss-es merged commit 50decdc into main Jul 29, 2024
36 checks passed
@ss-es ss-es deleted the ss/ungate-dependency-modules branch July 29, 2024 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants