Skip to content

Don't check in the Manifest#227

Merged
comnik merged 1 commit intomainfrom
ncg-remove-julia-manifest
Mar 6, 2026
Merged

Don't check in the Manifest#227
comnik merged 1 commit intomainfrom
ncg-remove-julia-manifest

Conversation

@comnik
Copy link
Collaborator

@comnik comnik commented Mar 6, 2026

The Julia community convention is:

  • Applications (deployed/reproducible environments): check in Manifest.toml for exact reproducibility.
  • Packages/libraries: do not check in Manifest.toml, because consumers will resolve their own dependency versions based on the [compat] bounds in Project.toml.

@comnik comnik self-assigned this Mar 6, 2026
Drvi
Drvi approved these changes Mar 6, 2026
View reviewed changes
Copy link
Contributor

@Drvi Drvi left a comment

Choose a reason for hiding this comment

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

LGTM!

@comnik comnik merged commit 992405f into main Mar 6, 2026
5 checks passed
@comnik comnik deleted the ncg-remove-julia-manifest branch March 6, 2026 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Drvi Drvi Drvi approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@comnik comnik

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@comnik @Drvi