Skip to content

fixed shaded package name#1743

Closed
lbergelson wants to merge 1 commit intogoogleapis:masterfrom
lbergelson:master
Closed

fixed shaded package name#1743
lbergelson wants to merge 1 commit intogoogleapis:masterfrom
lbergelson:master

Conversation

@lbergelson
Copy link
Copy Markdown
Contributor

Fixed the maven shade command so it produces a valid shaded package name. It was producing shaded.cloud-nio.* which is not a valid java package and causes compilation errors when trying to import those packages. This is problematic since the shaded packages are exposed in the API, so methods that exposed them were unusable by client code.

Shading now produces shading.cloud_nio.* which is valid

Fixed the maven shade command so it produces a valid shaded package name.  It was producing shaded.cloud-nio.* which is not a valid java package and causes compilation errors when trying to import those packages.  This is problematic since the shaded packages are exposed in the API, so methods that exposed them were unusable by client code.

Shading now produces shading.cloud_nio.* which is valid
@googlebot
Copy link
Copy Markdown

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Mar 13, 2017
@lbergelson
Copy link
Copy Markdown
Contributor Author

@jean-philippe-martin Could you take a look at this?

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 6173700 on lbergelson:master into ** on GoogleCloudPlatform:master**.

@jean-philippe-martin
Copy link
Copy Markdown

Looks good to me.

@garrettjonesgoogle
Copy link
Copy Markdown
Contributor

@lbergelson , could you do the CLA dance?

@lbergelson
Copy link
Copy Markdown
Contributor Author

@garrettjonesgoogle Sorry, I've been trying to talk to legal at the Broad. They've haven't responded to me yet. We have some sort of code sharing agreement already with google, but I'm not sure if they want me to sign the CLA or if there's some other agreement in place that takes precedent.

@garrettjonesgoogle
Copy link
Copy Markdown
Contributor

@jean-philippe-martin this change is pretty trivial - you could just make a different change that accomplishes the same thing and avoid the whole CLA issue.

@garrettjonesgoogle
Copy link
Copy Markdown
Contributor

Superseded by #1767

chingor13 pushed a commit that referenced this pull request Jan 22, 2026
chingor13 pushed a commit that referenced this pull request Feb 24, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
chingor13 pushed a commit that referenced this pull request Mar 12, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026
suztomo added a commit that referenced this pull request Mar 30, 2026
lqiu96 pushed a commit that referenced this pull request Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: no This human has *not* signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants