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(uniforms-patternfly): Port @kie-tools/uniforms-patternfly #143

Merged
merged 10 commits into from
Sep 25, 2023

Conversation

lordrip
Copy link
Member

@lordrip lordrip commented Sep 21, 2023

Context

The @kie-tools/uniforms-patternfly package depends on @patternfly v4.

Many packages from @kie-tools depend on @kie-tools/uniforms-patternfly provided from its workspace, this prevents upgrading it to @patternfly v5 since this will mean upgrading all dependencies at once.

Goal

This pull request aims to temporarily bring the @kie-tools/uniforms-patternfly package, make the necessary upgrades to @patternfly v5, and publish it under a different scope to provide an escape hatch for upgrading individual packages one by one while supporting the kaoto-next use cases.

Todo

  • Bring @kie-tools/uniforms-patternfly to kaoto-next
  • Make the tests run
  • Upgrade @patternfly to v5
  • Upgrade uniforms to v4
  • Handle the breaking changes
  • Ensure that all tests are passing again
  • Rename package, connect local version with kaoto-next

Todo for an upcoming pull request (https://github.com/KaotoIO/uniforms-patternfly)

  • Check for breaking changes -
  • Wire up versioning on every merge
  • Wire up publishing under the @kaoto-next scope, f.i. @kaoto-next/uniforms-patternfly
  • Upgrade the README.md to explain the goal of this port and what to expect next.
  • ⚠️ Optional enable eslint with the same config as the upstream repository

Notes

This pull request is a prerequisite for:

Related issues:

@lordrip lordrip added help wanted Extra attention is needed build Build and GitHub related issues labels Sep 21, 2023
@lordrip lordrip added this to the 0.1.0 - Smart Connectors' MVP milestone Sep 21, 2023
@lordrip lordrip self-assigned this Sep 21, 2023
@lordrip
Copy link
Member Author

lordrip commented Sep 22, 2023

At least one test is passing now 😄
image

@lordrip lordrip force-pushed the feat/uniforms-patternfly branch 3 times, most recently from 71586d1 to b03ff4e Compare September 23, 2023 19:10
@lordrip lordrip marked this pull request as ready for review September 23, 2023 19:27
@lordrip lordrip requested a review from a team September 23, 2023 19:27
@lordrip lordrip force-pushed the feat/uniforms-patternfly branch 2 times, most recently from d49bbc4 to 03fcbbe Compare September 23, 2023 20:19
@lordrip lordrip merged commit 46729b0 into KaotoIO:main Sep 25, 2023
5 checks passed
@lordrip lordrip deleted the feat/uniforms-patternfly branch September 25, 2023 09:23
@lhein lhein modified the milestones: Smart Connectors' MVP, Feature Parity Nov 2, 2023
lordrip pushed a commit to lordrip/kaoto that referenced this pull request Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Build and GitHub related issues help wanted Extra attention is needed
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants