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

feat: add manifest-to-markdown renderer #166

Conversation

paullatzelsperger
Copy link
Member

What this PR changes/adds

Adds a task doc2md, that takes the output JSON of autodoc and converts it into a Markdown file in the same directory (i.e. the build/ directory).

If there is a manifest.json in the root build/ dir, then that gets converted as well.

Why it does that

Easy and automatic generation of documentation.

Further notes

  • merging all the different edc.json files is async, so ./gradlew mergeManifest and ./gradlew doc2md must be separate gradle invocations
  • there is no dependency doc2md -> mergeManifest whatsoever

Linked Issue(s)

Closes # <-- insert Issue number if one exists

@paullatzelsperger paullatzelsperger force-pushed the feat/add_manifest_to_markdown_renderer branch from 3f40cd1 to 6464947 Compare July 31, 2023 14:45
@paullatzelsperger paullatzelsperger changed the title feat: add manifest to markdown renderer feat: add manifest-to-markdown renderer Jul 31, 2023
@paullatzelsperger paullatzelsperger added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 31, 2023
@paullatzelsperger paullatzelsperger force-pushed the feat/add_manifest_to_markdown_renderer branch 3 times, most recently from c249075 to aab982e Compare July 31, 2023 15:09

Verified

This commit was signed with the committer’s verified signature.
sandhose Quentin Gliech
@paullatzelsperger paullatzelsperger force-pushed the feat/add_manifest_to_markdown_renderer branch from aab982e to 87ff628 Compare July 31, 2023 15:10
@paullatzelsperger paullatzelsperger marked this pull request as draft July 31, 2023 15:12

Verified

This commit was signed with the committer’s verified signature.
sandhose Quentin Gliech

Verified

This commit was signed with the committer’s verified signature.
sandhose Quentin Gliech
@paullatzelsperger paullatzelsperger marked this pull request as ready for review July 31, 2023 15:18
* @param overview A string containing more information about the extension. Can be null, empty or even multiline.
* @param type The type of extension module, it can either be an SPI module or an implementation module
*/
void renderExtensionHeader(@NotNull String className, @Nullable String name, @Nullable String overview, ModuleType type);

Check notice

Code scanning / CodeQL

Useless parameter

The parameter 'type' is never used.
Copy link
Contributor

@wolf4ood wolf4ood left a comment

Choose a reason for hiding this comment

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

🚀 cool stuff :)

Just a comment on copyright headers

Verified

This commit was signed with the committer’s verified signature.
sandhose Quentin Gliech
@paullatzelsperger paullatzelsperger force-pushed the feat/add_manifest_to_markdown_renderer branch from 7ad1f95 to 30b0eb3 Compare July 31, 2023 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants