-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Fix minimize #10952
Fix minimize #10952
Conversation
The failing linter test is weird because that line that it is referencing has no code on it. |
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.
Don't worry about the lint failure, I'll fix that in master
if (options._calledWithOptions.minimize != null) { | ||
_minimize = options.minimize; | ||
} else if (defaultOptions.minimize != null) { | ||
_minimize = defaultOptions.minimize; | ||
} else if (options._parentOptions != null && options.getters != null) { |
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.
Why options.getters != null
here?
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.
it won't work without it. gh-9405 has tests already and this was the only combination that could get them all to pass.
I took a closer look and this is expected behavior. We made the change in #9405 that subdocs inherit parents' minimize and added tests that enforce that behavior, so no way to make this work. The If you find yourself struggling to reconcile behavior that's asserted on in the tests with new behavior, feel free to ask me to clarify in Slack. |
No description provided.