Skip to content
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(deps): update dependency com.google.protobuf:protoc to v3.22.2 #1525

Conversation

renovate-bot
Copy link
Contributor

@renovate-bot renovate-bot commented Feb 23, 2023

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
com.google.protobuf:protoc (source) 3.21.12 -> 3.22.2 age adoption passing confidence

Release Notes

protocolbuffers/protobuf

v3.22.2

v3.22.1

v3.22.0


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate-bot renovate-bot requested a review from a team as a code owner February 23, 2023 00:49
@forking-renovate forking-renovate bot added dependencies Pull requests that update a dependency file lang: java Issues specific to Java. labels Feb 23, 2023
@renovate-bot renovate-bot force-pushed the renovate/protocversion branch 13 times, most recently from 0fd219a to 3666b31 Compare February 23, 2023 20:28
@bourgeoisor
Copy link
Member

A handful of such errors:

/app/build/generated/source/proto/main/java/hipstershop/Demo.java:3672: error: emptyList() is not public in LazyStringArrayList; cannot be accessed from outside package
          com.google.protobuf.LazyStringArrayList.emptyList();
                                                 ^

@bourgeoisor
Copy link
Member

Potential help here: protocolbuffers/protobuf#11986

@renovate-bot renovate-bot force-pushed the renovate/protocversion branch 4 times, most recently from 27aeaeb to f50edd1 Compare March 1, 2023 13:54
@NimJay NimJay added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 1, 2023
@renovate-bot renovate-bot force-pushed the renovate/protocversion branch from f50edd1 to 2d63d76 Compare March 1, 2023 15:41
@NimJay NimJay removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 1, 2023
@renovate-bot renovate-bot force-pushed the renovate/protocversion branch 5 times, most recently from edaf72f to c700272 Compare March 2, 2023 17:01
@renovate-bot renovate-bot force-pushed the renovate/protocversion branch 3 times, most recently from 8fd9fb9 to 908f4d8 Compare March 27, 2023 17:46
@Shabirmean
Copy link
Member

Seems like we have to bump the correct dependency in the other services that generate the client stubs. I couldn't clearly tell which dependency installs protoc. I've tried a few things here: #1633.

@renovate-bot renovate-bot force-pushed the renovate/protocversion branch 13 times, most recently from f917f4d to 861b12f Compare April 2, 2023 06:35
@renovate-bot renovate-bot force-pushed the renovate/protocversion branch 4 times, most recently from 56595a9 to 2a3ee0f Compare April 11, 2023 16:55
@renovate-bot renovate-bot force-pushed the renovate/protocversion branch from 2a3ee0f to f89d322 Compare April 12, 2023 13:06
@renovate-bot renovate-bot force-pushed the renovate/protocversion branch from f89d322 to 515bf9f Compare April 13, 2023 04:08
@forking-renovate
Copy link

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

@bourgeoisor bourgeoisor mentioned this pull request Apr 21, 2023
@bourgeoisor
Copy link
Member

Closing in favour of #1718

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file lang: java Issues specific to Java.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants