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.

Sign up for GitHub

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

Jump to bottom

feat: pointers for multi level configuration #2283

Open
spiffcs opened this issue Nov 2, 2023 · 0 comments
Open

feat: pointers for multi level configuration #2283

spiffcs opened this issue Nov 2, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@spiffcs
Copy link
Contributor

spiffcs commented Nov 2, 2023 •
edited
Loading

What would you like to be added:

Syft's config options should be updated to use bool ptrs in order to allow for proper multi-level configuration. This allows for the distinction between false and unset values.

Why is this needed:
It's unclear if the default false zero value can behave correctly when trying to decide on values between config levels

Additional context:
See this comment from an earlier PR:
#2274 (comment)
@spiffcs spiffcs added the enhancement New feature or request label Nov 2, 2023
@spiffcs spiffcs mentioned this issue Nov 2, 2023
Merged
@spiffcs spiffcs changed the title Pointers for multi level configuration feat: pointers for multi level configuration Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@spiffcs