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

deps(ember): upgrade app to ember@4.12 + export of ember-modifier as direct dependency #57

Merged
merged 4 commits into from
Apr 27, 2023

Conversation

jayjayjpg
Copy link
Contributor

@jayjayjpg jayjayjpg commented Apr 26, 2023

Part of https://github.com/meroxa/platform-ui-v1/issues/881
Part of https://github.com/meroxa/product/issues/749
Closes #51

With this change, we're

  • upgrading mx-ui-components to the latest version ember@4.12
  • exporting ember-modifier as a direct dependency as required for the upgrade of our host application

Once this is merged and released, the upgrade work in https://github.com/meroxa/platform-ui-v1/issues/881 will be unblocked.

Demo

Chromatic Build: https://www.chromatic.com/build?appId=6109b9596e1c1300390c39c5&number=241

@changeset-bot
Copy link

changeset-bot bot commented Apr 26, 2023

🦋 Changeset detected

Latest commit: 812e151

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
mx-ui-components Minor

Not sure what this means? Click here to learn what changesets are.

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

@github-actions
Copy link
Contributor

Test coverage: 92.29%

🚨 No before coverage could be loaded

@github-actions
Copy link
Contributor

Test coverage: 92.29%

🚨 No before coverage could be loaded

1 similar comment
@github-actions
Copy link
Contributor

Test coverage: 92.29%

🚨 No before coverage could be loaded

Copy link

@janelletavares janelletavares left a comment

Choose a reason for hiding this comment

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

looks reasonable to me

"ember-keyboard": "8.2.0",
"ember-modifier": "^4.1.0",
Copy link
Member

Choose a reason for hiding this comment

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

this is nice

Copy link
Member

@raulb raulb left a comment

Choose a reason for hiding this comment

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

Love these changes.

options: {
singleQuote: true,
},
},
Copy link
Member

Choose a reason for hiding this comment

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

Nice addition.

@jayjayjpg jayjayjpg merged commit 40eb15b into main Apr 27, 2023
@jayjayjpg jayjayjpg deleted the jj/ember-upgrade-4.8 branch April 27, 2023 07:55
@meroxa-machine meroxa-machine mentioned this pull request Apr 27, 2023
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.

Ember Upgrade for mx-ui-components
4 participants