Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Upgrade highlight.js #6973

Merged
merged 3 commits into from
Oct 18, 2021
Merged

Upgrade highlight.js #6973

merged 3 commits into from
Oct 18, 2021

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Oct 18, 2021

Major version bump for highlight.js. Update usage of deprecated function & some cleanups.


This change is marked as an internal change (Task), so will not be included in the changelog.

Preview: https://616dc1587783b2097e463c31--matrix-react-sdk.netlify.app
⚠️ Do you trust the author of this PR? Maybe this build will steal your keys or give you malware. Exercise caution. Use test accounts.

Major version bump for highlight.js. Update usage of deprecated function & some cleanups.
@dbkr dbkr added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Oct 18, 2021
@dbkr dbkr requested a review from a team as a code owner October 18, 2021 18:34
dbkr added a commit to element-hq/element-web that referenced this pull request Oct 18, 2021
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

lgtm, though CI is unhappy with:

src/components/views/elements/SyntaxHighlight.tsx:18:10 - error TS2614: Module '"highlight.js"' has no exported member 'highlightBlock'. Did you mean to use 'import highlightBlock from "highlight.js"' instead?
--
  |  
  | 18 import { highlightBlock } from 'highlight.js';

@dbkr dbkr requested a review from turt2live October 18, 2021 18:50
@dbkr dbkr merged commit 98ba3fd into develop Oct 18, 2021
@dbkr dbkr deleted the dbkr/upgrade_highlightjs branch October 18, 2021 18:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants