-
Notifications
You must be signed in to change notification settings - Fork 2k
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
modify eventhub reademe #14778
modify eventhub reademe #14778
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally great, but if we follow the Link Guidelines:
- No locale info should be contained in the URL.
- We should not use relative URLs.
And I suggest that we put the URLs together at the bottom of README file which will make the content short and easy to edit.
sdk/spring/azure-spring-cloud-eventhubs-stream-binder/README.md
Outdated
Show resolved
Hide resolved
sdk/spring/azure-spring-boot-starter-active-directory-b2c/README.md
Outdated
Show resolved
Hide resolved
sdk/spring/azure-spring-cloud-eventhubs-stream-binder/README.md
Outdated
Show resolved
Hide resolved
sdk/spring/azure-spring-cloud-eventhubs-stream-binder/README.md
Outdated
Show resolved
Hide resolved
sdk/spring/azure-spring-cloud-starter-eventhubs-kafka/README.md
Outdated
Show resolved
Hide resolved
@saragluna If we seperate readme modification into multiple PRs, then will there be conflicts when we merge each of them? |
If each PR modifies a subset of README files then it should be fine. |
96276a2
to
e7fc76d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
modify README files of spring cloud eventhub modules for the release