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

Tweak how JavaDoc is converted into displayName/description #124

Merged
merged 4 commits into from
Dec 23, 2024

Conversation

timtebeek
Copy link
Contributor

What's changed?

Now set either description or displayName + description, but drop the longer generated description where possibe.

What's your motivation?

Previously we would often retain the generated Markdown that merely showed the input template, whereas there's JavaDoc comments that would be a better fit and make the docs more readable.

@timtebeek timtebeek self-assigned this Dec 23, 2024
@timtebeek timtebeek added the enhancement New feature or request label Dec 23, 2024
Comment on lines -85 to +88
return "Recipe created for the following Refaster template:\n```java\npublic static class NestedRule {\n \n @BeforeTemplate()\n String before(String s, String s1, String s2) {\n return s.replaceAll(s1, s2);\n }\n \n @AfterTemplate()\n String after(String s, String s1, String s2) {\n return s != null ? s.replaceAll(s1, s2) : s;\n }\n}\n```\n.";
return "A single line used as description";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This generated block greatly inflates our docs, which hurts readability. Better then to use the JavaDoc provided instead.

@timtebeek timtebeek merged commit dbee4fb into main Dec 23, 2024
2 checks passed
@timtebeek timtebeek deleted the tweak-how-javadoc-is-converted branch December 23, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants