-
Notifications
You must be signed in to change notification settings - Fork 1
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
[DCJ-616] Do not update action anymore #276
Conversation
After the changes in DataBiosphere/jade-data-repo#1781 and DataBiosphere/jade-data-repo#1794 , this workflow breaks the action
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.
👍🏽
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.
Thanks!
I'm a little confused that we are hitting this issue before the integration test refactor PR has been merged, can you help me see what I'm missing?
## Addresses https://broadworkbench.atlassian.net/browse/DCJ-616 ## Summary In TDR, Changes were recently made to the action which runs on PRs: - #1781 - #1794 The workflow in broadinstitute/datarepo-helm#276 tries to (unsuccessfully) modify the action, breaking it in the process: 31b4714 Links to failed builds: https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708679629 https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708678706 https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708668823 It added this little piece at the bottom which breaks the action.
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.
yay!!
…#1805) Update int-and-connected-test-run.yml ## Addresses https://broadworkbench.atlassian.net/browse/DCJ-616 ## Summary In TDR, Changes were recently made to the action which runs on PRs: - #1781 - #1794 The workflow in broadinstitute/datarepo-helm#276 tries to (unsuccessfully) modify the action, breaking it in the process: 31b4714 Links to failed builds: https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708679629 https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708678706 https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708668823 It added this little piece at the bottom which breaks the action.
Addresses
https://broadworkbench.atlassian.net/browse/DCJ-616
Summary
In TDR, Changes were recently made to the action which runs on PRs:
This workflow tries to (unsuccessfully) modify the action, breaking it in the process:
DataBiosphere/jade-data-repo@31b4714
Links to failed builds:
https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708679629
https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708678706
https://github.com/DataBiosphere/jade-data-repo/actions/runs/10708668823