-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
asciidoctor: 1.5.7.1 -> 2.0.10 #62707
Conversation
commit msg should be
|
There has been a new major version, so it seems good to upgrade. I had to drop two plugins: - `asciidoctor-latex` seems dead, and hasn't had any commits since 2015. - `asciidoctor-bespoke` also hasn't been updated for over a year, and doesn't seem to be compatible with the new version.
d0f51a3
to
3d1e0da
Compare
Commit message fixed. |
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.
I have not tested (not running a NixOS system currently) but it LGTM 👍 .
Also, small question: why did you remove asciidoctor-latex?
Ah, I said why I didn't feel bad about dropping it, but not why I had to: it specifies an old version of |
Looks good to me, is there someway I can use this nix expression to run their tests? |
No obviously. I tried installing rake and using |
Looks like that's WIP being fixed #62971 ✨ |
Do you want to wait to try that out or can we merge this? I've tested the executables by hand and they seem to work. |
That sounds fine, merging now. |
Motivation for this change
There has been a new major version, so it seems good to upgrade.
I had to drop two plugins:
asciidoctor-latex
seems dead, and hasn't had any commits since 2015.asciidoctor-bespoke
also hasn't been updated for over a year, anddoesn't seem to be compatible with the new version.
@yacinehmito
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)