Upgrade to Asciidoctor.js 2+

[ggrossetie]

Resolves #252

[obilodeau]

I would prefer the unrelated changes to be split out.

[obilodeau]

Note: I haven't tested it yet.

It's cool to get rid of the need to have users create their own converter script but to also keep it as an option with a stable API.

[ggrossetie]

Could you be more specific ? All changes are related to the upgrade 🤔

[obilodeau]

Could you be more specific ? All changes are related to the upgrade 🤔

I would prefer if the new syntax highlighter changes would be separate. I think this is what is breaking the tests right now.

[ggrossetie]

I would prefer if the new syntax highlighter changes would be separate. I think this is what is breaking the tests right now.

A test was failing when running with Asciidoctor.js: https://github.com/asciidoctor/asciidoctor-reveal.js/blob/master/examples/source-pygments.adoc

But you're right, I'm using the following (which is working with Asciidoctor.js but failing with Asciidoctor):

- linkcss = node.attr? 'linkcss'

I think it should be - linkcss = attr? 'linkcss'

[johnjelinek]

Is this good to merge?

[ggrossetie]

@obilodeau Please let me know if there's anything to fix before we can merge this upgrade 😃

[obilodeau]

I just did a new review. Only minor stuff, mostly documentation and comments.

[ggrossetie]

This pull request has too much changes, I will open pull request with smaller changes so it's easier for you to review.

[ggrossetie]

Now the documentation is Windows/PowerShell friendly.
I think it's ready to go.