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

[uniforms-patternfly] Upgrade @patternfly to v5.0.0 #1877

Open
lordrip opened this issue Jul 28, 2023 · 4 comments
Open

[uniforms-patternfly] Upgrade @patternfly to v5.0.0 #1877

lordrip opened this issue Jul 28, 2023 · 4 comments
Labels
area:core area:dependencies Pull requests that update a dependency file

Comments

@lordrip
Copy link
Contributor

lordrip commented Jul 28, 2023

Context

@patternfly has released its 5.0.0 version.

Upgrade information: https://staging.patternfly.org/get-started/upgrade

Goal

The goal for this issue is to upgrade @patternfly in the following repositories:

Notes

@kie-tools/uniforms-patternfly depends on:

    "@patternfly/react-core": "^4.276.6",
    "@patternfly/react-icons": "^4.93.6",

Related issue: KaotoIO/kaoto-ui#2209

@lordrip
Copy link
Contributor Author

lordrip commented Jul 28, 2023

There are the dependent packages from the repository
image

Taking this into account, what would be the best way to upgrade without breaking other package dependencies? Maybe create a uniforms-patternfly-1.0.0 branch and make the update? Later on link the dependency using branch resolution from yarn in the final consumers.

@yesamer yesamer added area:core area:dependencies Pull requests that update a dependency file labels Jul 31, 2023
@tiagobento
Copy link
Contributor

We usually upgrade for every package. PatternFly is extensively used on KIE Tools. Pretty much everywhere. Such upgrade is usually treated as a Major change. We even have a "Dependency consistency" check that runs to make sure all packages are using the same versions of dependencies.

There is, however, a way to bypass that. Adding an exception to https://github.com/kiegroup/kie-tools/blob/main/.syncpackrc.json will allow a package to not be aligned with others.

I don't recommend doing that, though, as in practice, packages that depend on uniforms-patternfly and are still using PF4 will be mixing imports. CSS will probably get messy, and this needs to be thoroughly tested.

We have PatternFly and React upgrades on the priority queue, though. We just can't provide an ETA for when it will land.

@lordrip
Copy link
Contributor Author

lordrip commented Aug 10, 2023

Sure thing @tiagobento, I agree with you about not mixing different @patternfly versions.

@lordrip
Copy link
Contributor Author

lordrip commented Oct 2, 2023

The library was temporarily ported to https://github.com/KaotoIO/uniforms-patternfly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:core area:dependencies Pull requests that update a dependency file
Projects
Status: 📋 Backlog
Development

No branches or pull requests

3 participants