-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Filter Synthetic Methods in PipelineOptionsFactory #41
Conversation
R: @lukecwik |
void setValue(Number value); | ||
} | ||
|
||
private static interface SubtypeReturingOpts extends Opts { |
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.
SubtypeReturingOpts -> SubtypeReturningOptions
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.
Done
Add a comment to PipelineOptions / PipelineOptionsFactory stating that we don't support java8's default methods. |
Opts -> Options for all class declarations Add a note that the value of a default method is ignored.
Use character replacement instead of regex for artifact flattening
* Suppress RandomModInteger Warnings Suppress RandomModInteger warnings since we are looking for a 1/n chance of being 0 as expected. https://errorprone.info/bugpattern/RandomModInteger * Increase Version
refactor: CoderUrnTree and dyn Coder
🤖 I have created a release \*beep\* \*boop\* --- ## [1.7.0](https://www.github.com/googleapis/python-firestore/compare/v1.6.2...v1.7.0) (2020-05-18) ### Features * Create CODEOWNERS ([apache#40](https://www.github.com/googleapis/python-firestore/issues/40)) ([a0cbf40](https://www.github.com/googleapis/python-firestore/commit/a0cbf403fe88f07c83bec81f275ac168be573e93)) ### Bug Fixes * **firestore:** fix get and getall method of transaction ([apache#16](https://www.github.com/googleapis/python-firestore/issues/16)) ([de3aca0](https://www.github.com/googleapis/python-firestore/commit/de3aca0e78b68f66eb76bc679c6e95b0746ad590)) * Update team to be in correct org ([apache#43](https://www.github.com/googleapis/python-firestore/issues/43)) ([bef5a3a](https://www.github.com/googleapis/python-firestore/commit/bef5a3af4613b5f9d753bb6f45275e480e4bb301)) * **firestore:** fix lint ([apache#48](https://www.github.com/googleapis/python-firestore/issues/48)) ([7fa00c4](https://www.github.com/googleapis/python-firestore/commit/7fa00c49dc3fab1d687fff9246f3e5ff0682cac0)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please).
Fixes the testReturnTypeConflcitThrows when compiled with java 8, which inserts a bridge method which is reported as a a conflicting type (even though it is an effect of a conflicting type rather than a cause, and the user cannot change anything about it)