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

Change groupId to com.azure and artifactId to azure-spring-cloud-starter-xxx #15771

Closed
wants to merge 19 commits into from
Closed

Conversation

ZhuXiaoBing-cn
Copy link
Contributor

update #13523

Xiaobing Zhu added 4 commits September 24, 2020 15:17
…to work

� Conflicts:
�	sdk/spring/azure-spring-boot-samples/azure-spring-boot-sample-cosmosdb/README.md
�	sdk/spring/azure-spring-boot-starter-cosmosdb/README.md
�	sdk/spring/azure-spring-cloud-servicebus-queue-stream-binder/README.md
@ghost ghost added the azure-spring All azure-spring related issues label Sep 28, 2020
[docs]: https://docs.microsoft.com/azure/developer/java/spring-framework/configure-spring-boot-starter-java-app-with-cosmos-db
[refdocs]: https://azure.github.io/azure-sdk-for-java/spring.html#azure-cosmosdb-spring-boot-starter
[package]: https://mvnrepository.com/artifact/com.microsoft.azure/azure-cosmosdb-spring-boot-starter
[sample]: https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/spring/azure-spring-boot-samples/azure-spring-boot-sample-cosmosdb
Copy link
Member

Choose a reason for hiding this comment

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

should sync code from azure master first

Copy link
Member

@moarychan moarychan left a comment

Choose a reason for hiding this comment

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

please comfirm with @wujack778 to avoid repeat modification.

Comment on lines +213 to +216
<include>com.azure:azure-spring-cloud-starter-context:[1.3.0-beta.1]</include> <!-- {x-include-update;com.azure:azure-spring-cloud-starter-context;current} -->
<include>com.azure:azure-spring-cloud-starter-telemetry:[1.3.0-beta.1]</include> <!-- {x-include-update;com.azure:azure-spring-cloud-starter-telemetry;current} -->
<include>com.azure:azure-spring-integration-eventhubs:[1.3.0-beta.1]</include> <!-- {x-include-update;com.azure:azure-spring-integration-eventhubs;current} -->
<include>com.azure:azure-spring-integration-servicebus:[1.3.0-beta.1]</include> <!-- {x-include-update;com.azure:azure-spring-integration-servicebus;current} -->
Copy link
Member

Choose a reason for hiding this comment

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

we can remove all the dependencies which the group id starts with com.azure:xxx in bannedDependencies part.

@moarychan
Copy link
Member

weidongxu-microsoft and others added 15 commits September 29, 2020 09:17
* fix sample VerifyNetworkPeeringWithNetworkWatcher

* enable VerifyNetworkPeeringWithNetworkWatcher
* mgmt update compute sample

* update script link
* Add tables module-info

* Add java-lts profile for tables

* Add unreleased dependency version for azure-storage-common
Fix missing JDK links in the track2 dataplane SDK READMEs
* Added OData filter expression helper class

* Unit tests for SearchFilter

* Fix linting issues
* Updating READMEs to have valid external links

* Update links to remove locales

* Fix README verify

* Fix build errors

* Fix build errors

* Update package links

* Update CHANGELOGs

* Address PR comments

* Address PR comments

Co-authored-by: Minnie Liu <[email protected]>
@ZhuXiaoBing-cn
Copy link
Contributor Author

I am sorry that there are some problems in my repository and there are redundant submissions. I need to close this PR now.

@chenrujun
Copy link

Hi, @ZhuXiaoBing-cn , you can use git push -f to avoid creating closed PR.

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
Development

Successfully merging this pull request may close these issues.