-
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
Publish the new binder package in releases. #8585
Conversation
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.
I do see some javadoc warnings that should be cleaned up (separate PR is fine).
> Task :grpc-binder:javadocs
grpc-java/binder/src/main/java/io/grpc/binder/internal/BinderTransport.java:129: warning - Tag @link: reference not found: EARLIEST_SUPPORTED_WIRE_FORMAT_VERSION
grpc-java/binder/src/main/java/io/grpc/binder/internal/BinderTransportSecurity.java:66: warning - Tag @link: reference not found: Attributes.Builder
grpc-java/binder/src/main/java/io/grpc/binder/ParcelableUtils.java:40: warning - Tag @link: reference not found: Metadata.Key
grpc-java/binder/src/main/java/io/grpc/binder/ParcelableUtils.java:53: warning - Tag @link: reference not found: Metadata.Key
grpc-java/binder/src/main/java/io/grpc/binder/ParcelableUtils.java:40: warning - Tag @link: reference not found: Metadata.Key
grpc-java/binder/src/main/java/io/grpc/binder/ParcelableUtils.java:53: warning - Tag @link: reference not found: Metadata.Key
grpc-java/binder/src/main/java/io/grpc/binder/ParcelableUtils.java:40: warning - Tag @link: reference not found: Metadata.Key
grpc-java/binder/src/main/java/io/grpc/binder/ParcelableUtils.java:53: warning - Tag @link: reference not found: Metadata.Key
8 warnings
Looks like the javadoc is including the internal directory. You probably want something like Lines 49 to 50 in 8ac9a4e
|
Re: Internal Javadoc, I tried excluding the internal directory, but that causes the whole task to fail (since the sources still depend on the internal package). It's not clear how to exclude the internal package from javadoc generation but not from the source path. |
Note: I didn't fix all javadoc warnings mentioned in grpc#8585, since they're not generated with a modern java version, and the fix feels worse than the warning. Specifically, {@link X.Y} generates a warning if only X is imported, and {@link Z} generates a warning if Z is declared later in the class. In particular, attempting to fix the first issue by importing X.Y results in a code-readability warning suggesting I shouldn't do that.
@markb74, that's what the |
Note: I didn't fix all javadoc warnings mentioned in #8585, since they're not generated with a modern java version, and the fix feels worse than the warning. Specifically, {@link X.Y} generates a warning if only X is imported, and {@link Z} generates a warning if Z is declared later in the class. In particular, attempting to fix the first issue by importing X.Y results in a code-readability warning suggesting I shouldn't do that.
No description provided.