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

feat(update to pf5): update dependencies to work with PF5 #93

Merged
merged 1 commit into from
Feb 12, 2024

Conversation

karelhala
Copy link
Contributor

Description

Since PF4 will soon not be supported we have to switch to PF5. This allows us to import dynamic components and layouts that way our chunks are not too big.

JIRA

https://issues.redhat.com/browse/RHCLOUD-30342

@karelhala karelhala requested a review from a team February 12, 2024 09:27
@karelhala karelhala force-pushed the update-to-pf5 branch 2 times, most recently from 071e148 to b7ddd20 Compare February 12, 2024 09:45
@fhlavac
Copy link

fhlavac commented Feb 12, 2024

Overall looks good, just some tests seem to be failing

@karelhala karelhala force-pushed the update-to-pf5 branch 2 times, most recently from 7587ec9 to 1420520 Compare February 12, 2024 10:41
@codecov-commenter
Copy link

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (d9fb7ac) 88.79% compared to head (5e75f47) 89.36%.
Report is 3 commits behind head on master.

Files Patch % Lines
...src/components/Formik/Patternfly/FormTextInput.tsx 66.66% 2 Missing ⚠️
...ript/src/components/Formik/Patternfly/Checkbox.tsx 75.00% 1 Missing ⚠️
...pt/src/components/Formik/Patternfly/FormSelect.tsx 75.00% 1 Missing ⚠️
...ript/src/components/Formik/Patternfly/FormText.tsx 66.66% 1 Missing ⚠️
.../src/components/Formik/Patternfly/FormTextArea.tsx 75.00% 1 Missing ⚠️
...script/src/components/Formik/Patternfly/Switch.tsx 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
+ Coverage   88.79%   89.36%   +0.56%     
==========================================
  Files          60       60              
  Lines         937      959      +22     
  Branches      166      181      +15     
==========================================
+ Hits          832      857      +25     
+ Misses        105      102       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@karelhala karelhala merged commit 4e20e52 into RedHatInsights:master Feb 12, 2024
1 check passed
@karelhala
Copy link
Contributor Author

@josejulio looks like the release failed. I guess I should do manual one, right?

@josejulio
Copy link
Member

@josejulio looks like the release failed. I guess I should do manual one, right?

That would be ok i guess. Very odd that it failed, probably some missing lint on a test file or similar.

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

Successfully merging this pull request may close these issues.

4 participants