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

Add flag for dry-run #22

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

sgasse
Copy link
Contributor

@sgasse sgasse commented Apr 30, 2024

With the dry-run capability, cargo-autoinherit can be run in CI and used
to enforce strict inheritance except for excluded workspace members.

The dry run will not touch any files in the workspace. However if any
file would be touched or there are any dependencies which could not be
autoinherited, it returns with a non-zero exit code.

sgasse added 2 commits April 30, 2024 17:42
There may be workspace members which have to stay in sync with other
workspaces and should be excluded from the auto-inheritance. This
commits allows to specify such workspace members by their package name.
With the dry-run capability, cargo-autoinherit can be run in CI and used
to enforce strict inheritance except for excluded workspace members.

The dry run will not touch any files in the workspace. However if any
file would be touched or there are any dependencies which could not be
autoinherited, it returns with a non-zero exit code.
@sgasse
Copy link
Contributor Author

sgasse commented Apr 30, 2024

Just saw this may be covered by #17

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.

1 participant