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

Refactor development dependency configuration #8309

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Oct 17, 2024

Part of #8090
Unblocks #8274

Refactors DevMode and DevSpecification into a shared type DevGroupsSpecification that allows us to track if --dev was implicitly or explicitly provided.

@zanieb zanieb added the internal A refactor or improvement that is not user-facing label Oct 17, 2024
@zanieb zanieb marked this pull request as ready for review October 17, 2024 21:52
Comment on lines +115 to +119
/// Return a new [`DevGroupsSpecification`] with development dependencies included by default.
///
/// This is appropriate in projects, where the `dev` group is synced by default.
#[must_use]
pub fn with_default_dev(self) -> Self {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will become with_default_groups or something in the future.

@zanieb zanieb force-pushed the zb/dev-refactor branch 2 times, most recently from 488eaa7 to 9048b37 Compare October 18, 2024 14:49
Copy link
Member

@charliermarsh charliermarsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice.

@zanieb zanieb merged commit e3f4abe into tracking/735 Oct 18, 2024
62 checks passed
@zanieb zanieb deleted the zb/dev-refactor branch October 18, 2024 16:58
zanieb added a commit that referenced this pull request Oct 18, 2024
Part of #8090 

Most of the heavy lifting is done in #8309
charliermarsh pushed a commit that referenced this pull request Oct 18, 2024
…8338)

Part of #8090 

Most of the heavy lifting is done in #8309 

Includes `--only-dev` which appears to be missing as an oversight.
charliermarsh pushed a commit that referenced this pull request Oct 20, 2024
Part of #8090 
Unblocks #8274

Refactors `DevMode` and `DevSpecification` into a shared type
`DevGroupsSpecification` that allows us to track if `--dev` was
implicitly or explicitly provided.
charliermarsh pushed a commit that referenced this pull request Oct 20, 2024
Part of #8090

Most of the heavy lifting is done in #8309
charliermarsh pushed a commit that referenced this pull request Oct 20, 2024
…8338)

Part of #8090

Most of the heavy lifting is done in #8309

Includes `--only-dev` which appears to be missing as an oversight.
charliermarsh pushed a commit that referenced this pull request Oct 22, 2024
Part of #8090 
Unblocks #8274

Refactors `DevMode` and `DevSpecification` into a shared type
`DevGroupsSpecification` that allows us to track if `--dev` was
implicitly or explicitly provided.
charliermarsh pushed a commit that referenced this pull request Oct 22, 2024
Part of #8090

Most of the heavy lifting is done in #8309
charliermarsh pushed a commit that referenced this pull request Oct 22, 2024
…8338)

Part of #8090

Most of the heavy lifting is done in #8309

Includes `--only-dev` which appears to be missing as an oversight.
charliermarsh pushed a commit that referenced this pull request Oct 22, 2024
Part of #8090 
Unblocks #8274

Refactors `DevMode` and `DevSpecification` into a shared type
`DevGroupsSpecification` that allows us to track if `--dev` was
implicitly or explicitly provided.
charliermarsh pushed a commit that referenced this pull request Oct 22, 2024
Part of #8090

Most of the heavy lifting is done in #8309
charliermarsh pushed a commit that referenced this pull request Oct 22, 2024
…8338)

Part of #8090

Most of the heavy lifting is done in #8309

Includes `--only-dev` which appears to be missing as an oversight.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal A refactor or improvement that is not user-facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants