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 Asciidoctor JS to 2.0.3, MathJaX to 2.7.6, HighlightJS to 9.16.2 #30

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

bric3
Copy link

@bric3 bric3 commented Nov 12, 2019

This PR is based on the bitbucket 6 support done in adds-bitbucket-6-support branch e.g. after #29.

The aim of this PR is to upgrade the component that allows to render asciidoc documents.

  • Asciidoctor.JS 2.0.3
  • MathJax to 2.76, also switching to new CDN as the previous one was decommissioned
  • HighlightJS 9.16.2, this library however was download with additional languages, see commit

Note that since the atlassian sdk is using YUI to compress javascript files, it doesn't work with modern JS and fails to compress asciidoctor.js, so the compression is disabled in the maven plugin, but the commits uses the minified versions that shipped with the original components libraries.

Note that to deploy the plugin using the atlassian sdk I had to use a final version not a snapshot one. I don't know how to bypass that.

Run the server `atlas-mvn bitbucket:run`, then
later on perform `atlas-mvn package`.
Note that since the atlassian sdk is using YUI to compress
javascript files, it doesn't work with modern JS and fails
to compress asciidoctor.js, so the compression is disabled
in the maven plugin, to work around it the plugin uses the
minified version that shipped with the original library.
Switching to new CDN as the previous one was decommissioned.
See https://www.mathjax.org/cdn-shutting-down/
POST https://highlightjs.org/download/
cookie: csrftoken=b5dklwK2x6n9MNBrR3JkuFuHyTtwbxMCD3FiPpQH3JBTySuPGp6j7iPsPTTwRoIz

csrfmiddlewaretoken=bLkk736HlhD5KtZKsxUmEAdFBuaRteM6DJMiBWcmRURPwyS8hThlhdyqSuAR95I3&apache.js=on&bash.js=on&cs.js=on&cpp.js=on&css.js=on&coffeescript.js=on&diff.js=on&go.js=on&xml.js=on&http.js=on&json.js=on&java.js=on&javascript.js=on&kotlin.js=on&less.js=on&lua.js=on&makefile.js=on&markdown.js=on&nginx.js=on&objectivec.js=on&php.js=on&perl.js=on&properties.js=on&python.js=on&ruby.js=on&rust.js=on&scss.js=on&sql.js=on&shell.js=on&swift.js=on&ini.js=on&typescript.js=on&yaml.js=on&plaintext.js=on&accesslog.js=on&ada.js=on&asciidoc.js=on&aspectj.js=on&abnf.js=on&bnf.js=on&cmake.js=on&clojure.js=on&dns.js=on&dart.js=on&dockerfile.js=on&elm.js=on&erlang.js=on&ebnf.js=on&golo.js=on&gradle.js=on&groovy.js=on&haskell.js=on&matlab.js=on&pgsql.js=on&powershell.js=on&protobuf.js=on&r.js=on&scala.js=on&tcl.js=on&thrift.js=on&vim.js=on
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