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

chore(deps): update shell to ui@9 #824

Merged
merged 2 commits into from
Mar 5, 2024
Merged

chore(deps): update shell to ui@9 #824

merged 2 commits into from
Mar 5, 2024

Conversation

Birkbjo
Copy link
Member

@Birkbjo Birkbjo commented Jan 16, 2024

Theoretically this should be a breaking change, since we update the major version of a dependency. However the changes in UI are not really changes that should affect people - just some removed proptypes - nevertheless they were part of the public API.

BREAKING CHANGE: Updated major version of UI

BREAKING CHANGE: Updated major version of UI
Copy link
Contributor

@KaiVandivier KaiVandivier left a comment

Choose a reason for hiding this comment

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

I think as-is, this would need to be released as a breaking change because it would actually break builds unless the consumer changes something (upgrades the rest of the app to ui@^9, uses a resolution, or uses the --no-verify tag) -- here's an example of an app with a dependency on ui@^8: (the "Yes" option from the prompt doesn't work either)

It may be an option to change how that package validation works though 🤷‍♂️

Screenshot 2024-01-26 at 5 07 37 PM

@Birkbjo
Copy link
Member Author

Birkbjo commented Jan 30, 2024

@KaiVandivier The commit has breaking change, so it would be a major version bump.

@tomzemp tomzemp merged commit b6a12fa into master Mar 5, 2024
6 checks passed
@tomzemp tomzemp deleted the chore/upgrade-ui-9 branch March 5, 2024 07:30
dhis2-bot added a commit that referenced this pull request Mar 5, 2024
# [11.0.0](v10.5.0...v11.0.0) (2024-03-05)

### chore

* **deps:** update shell to ui@9 ([#824](#824)) ([b6a12fa](b6a12fa))

### BREAKING CHANGES

* **deps:** Updated major version of UI
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 11.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

5 participants