-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add asciidoc module #102
Add asciidoc module #102
Conversation
|
||
public String apply(String asciidoc) { | ||
Document document = parser.parse(asciidoc, new Parser.ParserContext(null)); | ||
AsciidoctorLikeHtmlRenderer renderer = new AsciidoctorLikeHtmlRenderer(new AsciidoctorLikeHtmlRenderer.Configuration() |
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.
The renderer could be re-used for multiple files right?
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.
Yes, I will change it.
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.
The renderer could be re-used for multiple files right?
In that case, both Parser
and AsciidoctorLikeHtmlRenderer
should be thread-safe. Is that so?
asciidoc/runtime/src/main/resources/META-INF/quarkus-extension.yaml
Outdated
Show resolved
Hide resolved
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.
Added a few comments..
Tests are failing
@mcruzdev any progress on this? |
Hi @ia3andy, I'm resuming this now. |
@mcruzdev whats the dfiference in html you were concerned about ? |
...test/java/io/quarkiverse/qute/web/asciidoc/test/QuteAsciidocSectionWithInnerSectionTest.java
Outdated
Show resolved
Hide resolved
...yment/src/test/java/io/quarkiverse/qute/web/asciidoc/test/QuteAsciidocSectionHelperTest.java
Outdated
Show resolved
Hide resolved
@maxandersen, this library generates different semantic HTML tags compared to CommonMark. We need to submit some pull requests to the yupiik/tools-maven-plugin repository to ensure the behavior is as expected. While this isn’t an issue for standalone use, it can (we need to test) affect CSS application when used with Roq. |
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.
It looks good to me
@mkouba you have the final word.. |
...idoc/deployment/src/test/java/io/quarkiverse/qute/web/asciidoc/test/QuarkusAsciidocTest.java
Outdated
Show resolved
Hide resolved
...yment/src/test/java/io/quarkiverse/qute/web/asciidoc/test/QuteAsciidocSectionHelperTest.java
Outdated
Show resolved
Hide resolved
...test/java/io/quarkiverse/qute/web/asciidoc/test/QuteAsciidocSectionWithInnerSectionTest.java
Outdated
Show resolved
Hide resolved
asciidoc/runtime/src/main/resources/META-INF/quarkus-extension.yaml
Outdated
Show resolved
Hide resolved
asciidoc/runtime/src/main/resources/META-INF/quarkus-extension.yaml
Outdated
Show resolved
Hide resolved
Fix quarkiverse#66: RoqCollection by and group helpers
4b5778e
to
0e37a97
Compare
@mcruzdev thanks! |
Fixes #95