Skip to content

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
@zanieb zanieb requested a review from charliermarsh October 18, 2024 14:36
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
@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.
charliermarsh pushed a commit that referenced this pull request Oct 23, 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 23, 2024
Part of #8090

Most of the heavy lifting is done in #8309
charliermarsh pushed a commit that referenced this pull request Oct 23, 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 25, 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 25, 2024
Part of #8090

Most of the heavy lifting is done in #8309
charliermarsh pushed a commit that referenced this pull request Oct 25, 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 25, 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 25, 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.
zanieb added a commit that referenced this pull request Oct 25, 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 added a commit that referenced this pull request Oct 25, 2024
Part of #8090

Most of the heavy lifting is done in #8309
zanieb added a commit that referenced this pull request Oct 25, 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.

3 participants