-
Notifications
You must be signed in to change notification settings - Fork 37
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: bump the deps #462
chore: bump the deps #462
Conversation
Build succeeded. ✔️ pre-commit SUCCESS in 2m 29s |
Build succeeded. ✔️ pre-commit SUCCESS in 2m 30s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
.github/workflows/preview-pr.yml
Outdated
with: | ||
version: 9 | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we do install ourselves later on we can skip it here. Opened a PR #464 without realizing this tackled this too
with: | |
version: 9 | |
with: | |
version: 9 | |
run_install: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll drop the commit :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But I also had a look at the docs, and it looks like run_install
defaults to null
which is same as false
👀
Bump the dependencies as the broken page is caused by the change in the default behavior that's been fixed in the @patternfly/react-charts just 2 weeks ago. Fixes packit#461 Signed-off-by: Matej Focko <[email protected]>
882baea
to
171409d
Compare
Build succeeded. ✔️ pre-commit SUCCESS in 2m 10s |
Bump the dependencies as the broken usage page is caused by the change in the default behavior that's been fixed in the @patternfly/react-charts just 2 weeks ago.
Fixes #461