Skip to content

Update protos and tidy test code #848

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

Merged
merged 5 commits into from
Sep 25, 2024
Merged

Update protos and tidy test code #848

merged 5 commits into from
Sep 25, 2024

Conversation

chadlwilson
Copy link
Contributor

General maintenance

  • updates protos to latest, and ensures generation with matching grpc/protobuf versions
  • migrates the tests to junit jupiter consistently

@chadlwilson chadlwilson added the ReleaseCandidate If a PR is tagged with this label, after merging it will be released label Sep 25, 2024
@gaugebot
Copy link

gaugebot bot commented Sep 25, 2024

@chadlwilson Thank you for contributing to gauge-java. Your pull request has been labeled as a release candidate 🎉🎉.

Merging this PR will trigger a release.

Please bump up the version as part of this PR.

Instructions to bump the version can found at CONTRIBUTING.md

If the CONTRIBUTING.md file does not exist or does not include instructions about bumping up the version, please looks previous commits in git history to see what changes need to be done.

@chadlwilson
Copy link
Contributor Author

chadlwilson commented Sep 25, 2024

I don't understand why the tests are failing, to be honest - when they are fine locally. I must be missing something stupid.

OK, got it, GHA uses ancient Maven version with old base POM and we dont lock the surefire plugin version in POM. Will fix that.

@chadlwilson chadlwilson enabled auto-merge (squash) September 25, 2024 05:42
@chadlwilson chadlwilson merged commit a05e179 into master Sep 25, 2024
13 checks passed
@chadlwilson chadlwilson deleted the tidy-protoc-versions branch September 25, 2024 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ReleaseCandidate If a PR is tagged with this label, after merging it will be released
Development

Successfully merging this pull request may close these issues.

2 participants