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

fix: remove unused languages from codeeditor #136

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

ashtonG
Copy link
Contributor

@ashtonG ashtonG commented Sep 3, 2024

  • remove @uiw/codemirror-extensions-langs: Extra languages were not accessible and only served to bloat build size when consuming apps import the code editor by two megabytes.

  • use new yaml languagesupport: the old yaml package and the one included in the extension module did not support code folding.

  • reuse github flavored markdown instead of commonmark: github flavored markdown highlights regular links and tables.

* remove @uiw/codemirror-extensions-langs: Extra languages were not
  accessible and only served to bloat build size when consuming apps
  import the code editor

* use new yaml languagesupport: the old yaml package and the one
  included in the extension module did not support code folding.

* reuse github flavored markdown instead of commonmark: github flavored
  markdown highlights regular links and tables.
Copy link

netlify bot commented Sep 3, 2024

Deploy Preview for hew-ui ready!

Name Link
🔨 Latest commit f150486
🔍 Latest deploy log https://app.netlify.com/sites/hew-ui/deploys/66db165f3975a800088ed1c5
😎 Deploy Preview https://deploy-preview-136--hew-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@determined-ci
Copy link
Collaborator

determined-ci commented Sep 3, 2024

Hello! DesignKit diffs for commit 4e9fd39 are available for you to view here

@codecov-commenter
Copy link

codecov-commenter commented Sep 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.78%. Comparing base (870ce42) to head (4e9fd39).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #136   +/-   ##
=======================================
  Coverage   82.78%   82.78%           
=======================================
  Files          49       49           
  Lines        6482     6482           
  Branches      454      454           
=======================================
  Hits         5366     5366           
  Misses       1116     1116           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ashtonG ashtonG force-pushed the bug/remove-extra-languages branch from 4e9fd39 to f150486 Compare September 6, 2024 14:49
@ashtonG ashtonG merged commit 416c162 into main Sep 6, 2024
11 checks passed
@ashtonG ashtonG deleted the bug/remove-extra-languages branch September 6, 2024 14:51
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.

4 participants