-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Use Documentation Tags for Maven and Gradle in Getting Started #16234
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.
Thanks for the PR, @harpreets789! I've left some feedback inline.
Also, will you please change the commit to use Issue gh-16228
instead of Closes gh-16228
? That way the issue remains open for other documentation articles that need to introduce tabs.
Thank you for your thorough review and valuable suggestions. I appreciate the directions you've provided and will make the necessary changes accordingly. |
467dfdb
to
6fe76f5
Compare
…ation (spring-projects#16234) - Fixed minor formatting issues and addressed reviewer suggestions. - Issue spring-projectsgh-16228.
|
Thanks, @harpreets789! This is now merged into |
This PR consolidates the separate XML and Gradle code snippets into joint tabs within the getting-spring-security.adoc file
Issue gh-16228