-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upgrade to AsciidoctorJ v3 #17
base: master
Are you sure you want to change the base?
Conversation
Just a draft for now, until I get it fully working on my blog. |
8ecab28
to
0383379
Compare
Cool, just ping me when you're in a good place to merge and I can do a release. |
Upgrade to the latest AsciidoctorJ v3.0. Also, add deps.edn for easier inclusion in deps-based projects. Bump version of the plugin itself to 1 - it has been stable for a long time. Key code changes: * Use the OptionsBuilder -> Options (though as little as possible) instead of the deprecated map argument. * When cryogen's `:debug?` is on, print the final options - useful in troubleshooting
Status: My blog builds fine, with only small differences in something that is anyway invalid asciidoc. |
This is key internal to cryogen-asciidoc and of no interest to AsciidoctorJ.
Actually, it seems to be working fine now, and my Anyway, it may be best to wait with this PR until an actual release of AsciidoctorJ v3. |
Sure thing, code wise everything looks good and we can just wait for the release then to merge this. |
Upgrade to the latest AsciidoctorJ v3.0.
Also, add deps.edn for easier inclusion in deps-based projects.
Bump version of the plugin itself to 1 - it has been stable for a long time.
Key code changes:
deprecated map argument.
:debug?
is on, print the final options - useful in troubleshootingSee https://github.com/asciidoctor/asciidoctorj/blob/main/CHANGELOG.adoc for more info. There are some minor breaking changes compared to v2.5, the main one being likely that Java >=11 is not required.