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

Upgrade to AsciidoctorJ v3 #17

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Upgrade to AsciidoctorJ v3 #17

wants to merge 2 commits into from

Conversation

holyjak
Copy link

@holyjak holyjak commented Nov 22, 2023

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

See 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.

@holyjak holyjak marked this pull request as draft November 22, 2023 01:07
@holyjak
Copy link
Author

holyjak commented Nov 22, 2023

Just a draft for now, until I get it fully working on my blog.

@holyjak holyjak force-pushed the adoc-v3 branch 4 times, most recently from 8ecab28 to 0383379 Compare November 22, 2023 01:22
@yogthos
Copy link
Member

yogthos commented Nov 22, 2023

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
@holyjak
Copy link
Author

holyjak commented Nov 22, 2023

Status: My blog builds fine, with only small differences in something that is anyway invalid asciidoc. But support for custom macros is broken (i.e. they are not processed), investigating...

This is key internal to cryogen-asciidoc and of no interest to AsciidoctorJ.
@holyjak
Copy link
Author

holyjak commented Nov 22, 2023

Actually, it seems to be working fine now, and my abbr macro gets proerly expanded. Not sure what was the issue.

Anyway, it may be best to wait with this PR until an actual release of AsciidoctorJ v3.

@yogthos
Copy link
Member

yogthos commented Nov 22, 2023

Sure thing, code wise everything looks good and we can just wait for the release then to merge this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants