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

Merged / Stack Config #541

Open
dsully opened this issue Jul 31, 2023 · 0 comments
Open

Merged / Stack Config #541

dsully opened this issue Jul 31, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@dsully
Copy link
Contributor

dsully commented Jul 31, 2023

Is your feature request related to a problem? Please describe.
I have a need (in a corporate environment) for a global list of allow + deny list, but also the ability to add to the exclusion list on a per-repo basis.

Describe the solution you'd like
Two different approaches I can think of:

  1. Allow passing --config multiple times and merge the configuration (order might matter here?)

  2. Allow for passing --config but also look for <cwd>/deny.toml (and the other locations in pub fn get_config_path and do the config merge with the local key/values having precedence.

Changes to ValidConfig.load() would need to be made in either case.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
I can put together a PR if this is functionality that would be accepted.

Thanks!

@dsully dsully added the enhancement New feature or request label Jul 31, 2023
dsully added a commit to dsully/cargo-deny that referenced this issue Aug 4, 2023
Implement a solution for additional license exceptions configuration
as described in EmbarkStudios#541
Jake-Shadle pushed a commit to dsully/cargo-deny that referenced this issue Sep 3, 2023
Implement a solution for additional license exceptions configuration
as described in EmbarkStudios#541
mergify bot pushed a commit that referenced this issue Sep 3, 2023
Implement a solution for additional license exceptions configuration as
described in #541

Happy to entertain other solutions or approaches.

Minor typo fixed as well.

---------

Co-authored-by: Jake Shadle <jake.shadle@embark-studios.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant