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 major devDependenices #2624

Open
wants to merge 29 commits into
base: main
Choose a base branch
from
Open

Conversation

dchyun
Copy link
Contributor

@dchyun dchyun commented Dec 23, 2024

๐Ÿ“Œ Summary

If merged, this PR would upgrade most (exceptions detailed below) devDependencies to the latest major versions, and remove jsdom from the website devDependencies.

๐Ÿ› ๏ธ Detailed description

These are upgrades for our tooling dependencies and do not require releases for our packages.

The intent is to merge this PR following the other upgrades listed below to avoid significant conflicts.

Exceptions:

  • ember-cli - Will be addressed in a separate update where the ember-cli blueprint is updated as well
  • ember-prism - Package is marked as archived, and alternatives will be researched separately
  • svgo / svgr - related dependencies (we made a conscious decision to use v 1.x)
  • @ember/test-waiters, @embroider/addon-dev, @embroider/test-setup, @glimmer/component, algoriasearch, del, ember-qunit, eslint
    • Non-trivial code changes required for upgrades. Will be addressed in separate PRs.
  • ember-a11y-refocus, ember-modifier, sass
    • Package updates will be handled in upcoming updates to components packages dependencies

๐Ÿ”— External links

Jira ticket: HDS-4251


๐Ÿ‘€ Component checklist

  • Percy was checked for any visual regression

๐Ÿ’ฌ Please consider using conventional comments when reviewing this PR.

Copy link

vercel bot commented Dec 23, 2024

The latest updates on your projects. Learn more about Vercel for Git โ†—๏ธŽ

Name Status Preview Updated (UTC)
hds-showcase โœ… Ready (Inspect) Visit Preview Dec 23, 2024 9:04pm
hds-website โœ… Ready (Inspect) Visit Preview Dec 23, 2024 9:04pm

Copy link
Collaborator

@Dhaulagiri Dhaulagiri left a comment

Choose a reason for hiding this comment

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

left a couple of small points of feedback for us to consider

website/package.json Outdated Show resolved Hide resolved
showcase/package.json Outdated Show resolved Hide resolved
"ember-meta": "^2.0.0",
"ember-modifier": "^4.1.0",
"ember-page-title": "^8.2.3",
"ember-power-select": "^8.2.0",
"ember-prism": "^0.13.0",
"ember-prism": "^1.0.0",
Copy link
Collaborator

@Dhaulagiri Dhaulagiri Dec 23, 2024

Choose a reason for hiding this comment

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

noticed that this package is now archived, so we should create a ticket in our backlog to investigate other options...

Copy link
Collaborator

Choose a reason for hiding this comment

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

here is better alternative I recently used to replace ember-prism https://github.com/IgnaceMaes/ember-shiki

Copy link
Collaborator

Choose a reason for hiding this comment

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

I could have been more clear with my comment: I think the upgrade is fine but down the road we'll need to look at other options potentially

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants