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(codegen): remove version replacement for modules. #1353

Merged
merged 4 commits into from
Nov 23, 2022
Merged

fix(codegen): remove version replacement for modules. #1353

merged 4 commits into from
Nov 23, 2022

Conversation

zhumin8
Copy link
Contributor

@zhumin8 zhumin8 commented Nov 21, 2022

Using placeholder for parent version for unblocking generation workflow setup. For future improvements:

  • Use revision to get versions from spring-cloud-gcp parent pom. This will involve changes for all modules with spring-cloud-gcp as parent.

@zhumin8 zhumin8 marked this pull request as ready for review November 23, 2022 16:43
Copy link
Contributor

@diegomarquezp diegomarquezp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

generated/pom.xml Outdated Show resolved Hide resolved
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@zhumin8 zhumin8 requested a review from emmileaf November 23, 2022 18:53
zhumin8 added a commit to googleapis/sdk-platform-java that referenced this pull request Nov 23, 2022
- `{{starter-version}}` are not needed anymore as we inherit version from parent pom.
- add parent version placeholder. Changes on spring-cloud-gcp side: GoogleCloudPlatform/spring-cloud-gcp#1353.
- remove redundant groupid.
- remove versions for `spring-cloud-gcp-core` and `spring-boot-starter`, specified in `genereated-parent` pom
@zhumin8 zhumin8 merged commit 4898740 into main Nov 23, 2022
@zhumin8 zhumin8 deleted the fix-pom branch November 23, 2022 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants