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

In azure-spring-boot-bom, change certificate-starter's version to "${project.version}". #24734

Conversation

chenrujun
Copy link

@chenrujun chenrujun commented Oct 12, 2021

Fix #24459.

  1. In azure-spring-boot-bom, change certificate-starter's version to "${project.version}".
  2. Use azure-spring-boot-bom in certificate-starter's README.md.

…"${project.version}".

2. Fix version error in certificate-starter's README.md.
@ghost ghost added the azure-spring All azure-spring related issues label Oct 12, 2021
@chenrujun chenrujun self-assigned this Oct 12, 2021
@chenrujun chenrujun added this to the [2021] November milestone Oct 12, 2021
@@ -92,7 +91,7 @@
<dependency>
<groupId>com.azure.spring</groupId>
<artifactId>azure-spring-boot-starter-keyvault-certificates</artifactId>
<version>${azure.spring.boot.starter.keyvault.certificates.version}</version>
<version>${project.version}</version>
Copy link
Author

Choose a reason for hiding this comment

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

Hi, @mnriem .

FYI: If appconfiguration related modules' version same to other spring-cloud-azure-xxx, our bom will be much simplier.

If you don't want to change in main branch, could you please change in 4.0 branch?

@chenrujun chenrujun linked an issue Oct 12, 2021 that may be closed by this pull request
@chenrujun chenrujun changed the title 1. In azure-spring-boot-bom, change certificate-starter's version to "${project.version}". In azure-spring-boot-bom, change certificate-starter's version to "${project.version}". Oct 12, 2021
Copy link
Member

@yiliuTo yiliuTo left a comment

Choose a reason for hiding this comment

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

LGTM

@chenrujun chenrujun merged commit 0a13646 into Azure:main Oct 12, 2021
@chenrujun chenrujun deleted the In_azure-spring-boot-bom_change_certificate_starter_version_to_project_version branch October 12, 2021 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
azure-spring All azure-spring related issues
Projects
None yet
3 participants