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

bump scalprum #2645

Merged
merged 2 commits into from
Oct 10, 2023
Merged

Conversation

Hyperkid123
Copy link
Contributor

@codecov-commenter
Copy link

codecov-commenter commented Oct 5, 2023

Codecov Report

Merging #2645 (8320a9a) into master (6d9c5d8) will not change coverage.
The diff coverage is n/a.

❗ Current head 8320a9a differs from pull request most recent head 7b14a42. Consider uploading reports for the commit 7b14a42 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2645   +/-   ##
=======================================
  Coverage   57.47%   57.47%           
=======================================
  Files          93       93           
  Lines        2843     2843           
  Branches      662      582   -80     
=======================================
  Hits         1634     1634           
- Misses       1134     1208   +74     
+ Partials       75        1   -74     

see 25 files with indirect coverage changes

@Hyperkid123 Hyperkid123 force-pushed the scalprum-update branch 2 times, most recently from 9822b4d to d8b8a15 Compare October 5, 2023 12:38
@Hyperkid123 Hyperkid123 merged commit dce730b into RedHatInsights:master Oct 10, 2023
@Hyperkid123 Hyperkid123 deleted the scalprum-update branch October 10, 2023 11:05
@@ -127,7 +127,7 @@
"@data-driven-forms/pf4-component-mapper": "^3.20.2",
"@data-driven-forms/react-form-renderer": "^3.20.2",
"@formatjs/cli": "4.8.4",
"@openshift/dynamic-plugin-sdk": "^3.0.0",
"@openshift/dynamic-plugin-sdk": "^4.0.0",
Copy link
Contributor

@florkbr florkbr Oct 10, 2023

Choose a reason for hiding this comment

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

@Hyperkid123 should we also bump the SDK in the dev-dependencies section? EDIT - nevermind - that is the webpack package

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