-
Notifications
You must be signed in to change notification settings - Fork 20
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(recordings): remove subdirectory for jvmId in recording requests #1127
chore(recordings): remove subdirectory for jvmId in recording requests #1127
Conversation
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.
Looks good! Just need to run yarn format:apply && yarn eslint:apply
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.
Looks good to me
c259066
to
9a7ad48
Compare
6d91d25
to
9a7ad48
Compare
This PR/issue depends on:
|
5469545
to
41c6eeb
Compare
/build_test |
Test image available:
|
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.
Rebase please
3f15604
to
69aa679
Compare
huh after rebasing, it's failing 90% of tests when I run |
|
yea that fixed it - I think it was the |
69aa679
to
a5ee6d7
Compare
#1127) (#1150) (cherry picked from commit bf9af76) Co-authored-by: Ming Yu Wang <[email protected]>
Welcome to Cryostat! 👋
Before contributing, make sure you have:
main
branch[chore, ci, docs, feat, fix, test]
To recreate commits with GPG signature
git fetch upstream && git rebase --force --gpg-sign upstream/main
Fixes: #993
Depends on https://github.com/cryostatio/cryostat/pull/1692
Depends on https://github.com/cryostatio/cryostat/issues/1738