-
Notifications
You must be signed in to change notification settings - Fork 244
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
BSR-704/default except override for optimize_for #1625
BSR-704/default except override for optimize_for #1625
Conversation
…add a testing helper
private/buf/bufgen/generator.go
Outdated
optimizeFor, err := bufimagemodify.OptimizeFor( | ||
logger, | ||
sweeper, | ||
*managedConfig.OptimizeFor, | ||
managedConfig.OptimizeForConfig.Default, | ||
managedConfig.JavaPackagePrefix.Except, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be OptimizeForConfig.Except
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for catching this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good -- will need to fix the merge conflict in the changelog, since a release went out recently, but otherwise it seems fine. Thanks!
* Add default, except override to managed mode `optimize_for` option * preserves backward compatibility with the current `optimize_for` option being string
optimize_for
optionoptimize_for
option being string