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

Bump highlight.js from 10.7.3 to 11.3.1 #1942

Closed
wants to merge 1 commit into from
Closed

Conversation

azul
Copy link
Contributor

@azul azul commented Nov 9, 2021

Use path to the node_modules folder to work around
webpack/webpack#13865

Bumps highlight.js from 10.7.3 to 11.3.1.


updated-dependencies:

  • dependency-name: highlight.js
    dependency-type: direct:production
    update-type: version-update:semver-major
    ...

Signed-off-by: dependabot[bot] [email protected]

  • Resolves: #
  • Target version: master

Summary

@azul azul force-pushed the azul/update-highlight.js branch 2 times, most recently from a14918f to 9801291 Compare November 9, 2021 22:05
Use path to the node_modules folder to work around
webpack/webpack#13865

Bumps [highlight.js](https://github.com/highlightjs/highlight.js) from 10.7.3 to 11.3.1.
- [Release notes](https://github.com/highlightjs/highlight.js/releases)
- [Changelog](https://github.com/highlightjs/highlight.js/blob/main/CHANGES.md)
- [Commits](highlightjs/highlight.js@10.7.3...11.3.1)

---
updated-dependencies:
- dependency-name: highlight.js
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@azul azul force-pushed the azul/update-highlight.js branch from 9801291 to bb991a8 Compare November 9, 2021 22:21
Comment on lines -51 to +53
const lang = await import(/* webpackChunkName: "highlight/[request]" */'highlight.js/lib/languages/' + languages[i])
const lang = await import(
/* webpackChunkName: "highlight/[request]" */
'../node_modules/highlight.js/lib/languages/' + languages[i])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the actual workaround.
Not sure if that's viable for now.

@azul azul requested a review from juliusknorr November 9, 2021 22:55
@juliusknorr
Copy link
Member

juliusknorr commented Nov 10, 2021

Nice, seems to do the trick for the import 🎉

However when doing some further tests the Highlight plugin of tiptap-extensions seems to be bound to lowlight 1.x which is not compatible with highlight.js 11.x so the highlighting is still not working yet. Maybe manually pinning lowlight to 2.4 works fine.

@azul
Copy link
Contributor Author

azul commented Nov 10, 2021

@juliushaertl good catch!

I think we should just stick to highlight.js 10 series then and look into upgrading to tiptap 2 etc.
I'd also be up to propose a PR against tiptap v1 - but it has not seen any releases since march and i doubt they will do dependency update releases.

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