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 duplicate slashes in URL links #21108
Fix duplicate slashes in URL links #21108
Conversation
8f0d626
to
a87a904
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.
Thank you very much for raising your first PR @jkatada
Rather that removing the /
after the variable, I'd like us to add it (if necessary) and remove it from the variable. We can see that's what we've done for code there and it reads more naturally to me.
Would you have time to rework the PR in that direction?
@@ -91,7 +91,7 @@ private void writeTable(AutoConfiguration autoConfigurationClasses) throws IOExc | |||
writer.println(); | |||
writer.printf("| {spring-boot-code}/spring-boot-project/%s/src/main/java/%s.java[`%s`]%n", | |||
autoConfigurationClasses.module, autoConfigurationClass.path, autoConfigurationClass.name); | |||
writer.printf("| {spring-boot-api}/%s.html[javadoc]%n", autoConfigurationClass.path); | |||
writer.printf("| {spring-boot-api}%s.html[javadoc]%n", autoConfigurationClass.path); |
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.
I think we should rather stick to a /
after the variable rather than no slash. This means that {spring-boot-api}
would not end with a slash.
a87a904
to
3f8c56e
Compare
@snicoll Thanks for the feedback. I've updated the PR. Please review. |
@jkatada thanks for the follow-up and for making your first contribution to Spring Boot. |
Fix links which have duplicate slashes in reference documentation.
link examples