Skip to content

Migrate to Asciidoctor 2.0.11 #1554

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

Merged
merged 1 commit into from
Nov 6, 2020
Merged

Migrate to Asciidoctor 2.0.11 #1554

merged 1 commit into from
Nov 6, 2020

Conversation

HonkingGoose
Copy link
Contributor

Changes

  • Bump Asciidoctor from 1.5.6.2 to 2.0.11.
  • Change style to new specifications.
  • Pin dependencies.
  • Fix unintended highlighting of text.
  • Undo rake script on Markdown file.
  • Undo rake script on Ruby file.

Context

Fixes #1553
Fixes #1541
Fixes #1358
Fixes #1373

- Bump Asciidoctor from 1.5.6.2 to 2.0.11
- Change style to new specifications.
- Pin dependencies
- Fix unintended highlighting of text
- Undo rake script on Markdown file
- Undo rake script on Ruby file

Co-authored-by: HonkingGoose <[email protected]>
@HonkingGoose
Copy link
Contributor Author

I've squashed the changes that I and @jnavila made on #1373, and also merged #1541 into this branch.
I've also updated to the latest version of Asciidoctor (2.0.11) which seems to work just fine.

I manually migrated some missing legacy double quoted syntax to the modern syntax.

I think this is ready to go, but @jnavila can you please run that migration script again on this branch, and see what it catches?

@ben can you take a look at this and tell me what you think of it?

Copy link
Member

@ben ben left a comment

Choose a reason for hiding this comment

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

LGTM 👍 You may merge when ready. 😁

@jnavila
Copy link
Member

jnavila commented Nov 6, 2020

This was mostly a quoting issue. So OK for me.

@jnavila jnavila merged commit 0042bf2 into progit:master Nov 6, 2020
@HonkingGoose
Copy link
Contributor Author

Woohoo! 🥳 🎈 🎆 ✨

@HonkingGoose
Copy link
Contributor Author

By the way the project board that @jnavila made on the progit2 repo can be closed/renamed/deleted as we no longer need it for the migration. 😉

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.

Proposal: use versions for other Ruby gems too.
3 participants