Skip to content

Comments

chore: use only highlight.js 11.8.0#35572

Merged
ggazzo merged 7 commits intodevelopfrom
chore/highlight-update
Apr 30, 2025
Merged

chore: use only highlight.js 11.8.0#35572
ggazzo merged 7 commits intodevelopfrom
chore/highlight-update

Conversation

@MartinSchoeler
Copy link
Member

@MartinSchoeler MartinSchoeler commented Mar 20, 2025

Proposed changes (including videos or screenshots)

Stops using two different versions of highlight.js, instead uses only the latest one

Issue(s)

https://rocketchat.atlassian.net/browse/CORE-1031

Steps to test or reproduce

Send messages with code quotes with code from different languages

Further comments


This pull request updates the highlight.js library in the Rocket.Chat repository from version 9 to 11.8.0. The changes include updating the import paths and language registration logic in apps/meteor/app/markdown/lib/hljs.js to improve maintainability and compatibility with the latest version of highlight.js. Additionally, the import statement for highlight.js styles in apps/meteor/client/startup/startup.ts is updated from 'hljs9/styles/github.css' to 'highlight.js/styles/github.css' to ensure correct dependency usage.

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Mar 20, 2025

Looks like this PR is not ready to merge, because of the following issues:

  • This PR is targeting the wrong base branch. It should target 7.7.0, but it targets 7.6.0

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

@changeset-bot
Copy link

changeset-bot bot commented Mar 20, 2025

⚠️ No Changeset found

Latest commit: 2ce3816

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Mar 20, 2025

PR Preview Action v1.6.1

🚀 View preview at
https://RocketChat.github.io/Rocket.Chat/pr-preview/pr-35572/

Built to branch gh-pages at 2025-04-29 20:27 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@MartinSchoeler MartinSchoeler added this to the 7.7.0 milestone Apr 17, 2025
@codecov
Copy link

codecov bot commented Apr 17, 2025

Codecov Report

Attention: Patch coverage is 10.98266% with 154 lines in your changes missing coverage. Please review.

Project coverage is 61.14%. Comparing base (8ba005e) to head (2ce3816).
Report is 59 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #35572      +/-   ##
===========================================
- Coverage    61.17%   61.14%   -0.04%     
===========================================
  Files         3005     3013       +8     
  Lines        71387    71594     +207     
  Branches     16342    16382      +40     
===========================================
+ Hits         43669    43773     +104     
- Misses       24750    24848      +98     
- Partials      2968     2973       +5     
Flag Coverage Δ
e2e 57.83% <8.87%> (-0.04%) ⬇️
unit 75.11% <2.89%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kody-ai
Copy link

kody-ai bot commented Apr 22, 2025

Code Review Completed! 🔥

The code review was successfully completed based on your current configurations.

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Security
Code Style
Kody Rules
Refactoring
Error Handling
Maintainability
Potential Issues
Documentation And Comments
Performance And Optimization
Breaking Changes

Access your configuration settings here.

Copy link
Member

@julio-rocketchat julio-rocketchat left a comment

Choose a reason for hiding this comment

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

Security-wise LGTM. Thanks a lot, @MartinSchoeler

@MartinSchoeler MartinSchoeler marked this pull request as ready for review April 23, 2025 12:03
@MartinSchoeler MartinSchoeler requested review from a team as code owners April 23, 2025 12:03
Copy link
Member

@gabriellsh gabriellsh left a comment

Choose a reason for hiding this comment

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

Colors slightly changed (original on top). I don't think this is a huge deal but could you investigate and/or talk to the product team to discuss this? I also wonder if this should be a chore since the styles are changing.
image
image

@MartinSchoeler MartinSchoeler changed the title chore: update hljs9 to highlight.js 11.8.0 fix: update hljs9 to highlight.js 11.8.0 Apr 23, 2025
@MartinSchoeler MartinSchoeler changed the title fix: update hljs9 to highlight.js 11.8.0 fix: use only highlight.js 11.8.0 Apr 23, 2025
@MartinSchoeler
Copy link
Member Author

Moving this to fix, since the update change some colors

@kody-ai
Copy link

kody-ai bot commented Apr 23, 2025

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Security
Code Style
Kody Rules
Refactoring
Error Handling
Maintainability
Potential Issues
Documentation And Comments
Performance And Optimization
Breaking Changes

Access your configuration settings here.

@kody-ai
Copy link

kody-ai bot commented Apr 24, 2025

Code Review Completed! 🔥

The code review was successfully completed based on your current configurations.

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Security
Code Style
Kody Rules
Refactoring
Error Handling
Maintainability
Potential Issues
Documentation And Comments
Performance And Optimization
Breaking Changes

Access your configuration settings here.

@MartinSchoeler MartinSchoeler changed the title fix: use only highlight.js 11.8.0 chore: use only highlight.js 11.8.0 Apr 28, 2025
@kody-ai
Copy link

kody-ai bot commented Apr 28, 2025

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Security
Code Style
Kody Rules
Refactoring
Error Handling
Maintainability
Potential Issues
Documentation And Comments
Performance And Optimization
Breaking Changes

Access your configuration settings here.

@scuciatto scuciatto added the stat: QA assured Means it has been tested and approved by a company insider label Apr 29, 2025
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Apr 29, 2025
@kody-ai
Copy link

kody-ai bot commented Apr 29, 2025

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Security
Code Style
Kody Rules
Refactoring
Error Handling
Maintainability
Potential Issues
Documentation And Comments
Performance And Optimization
Breaking Changes

Access your configuration settings here.

@kody-ai
Copy link

kody-ai bot commented Apr 29, 2025

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Security
Code Style
Kody Rules
Refactoring
Error Handling
Maintainability
Potential Issues
Documentation And Comments
Performance And Optimization
Breaking Changes

Access your configuration settings here.

@ggazzo ggazzo merged commit eaad62b into develop Apr 30, 2025
46 of 48 checks passed
@ggazzo ggazzo deleted the chore/highlight-update branch April 30, 2025 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants