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

Workflows: fix test-all-scream action syntax #7006

Merged
merged 2 commits into from
Feb 13, 2025

Conversation

bartgol
Copy link
Contributor

@bartgol bartgol commented Feb 13, 2025

This should fix the eamxx-sa workflow, which has not been running correctly in the nightlies.

Also, run eamxx-sa-testing workflow when the test-all-scream action is updated.

[BFB]


* We cannot use an input unless we declare it first.
* Fix bash var append syntax

Signed-off-by: Luca Bertagna <[email protected]>
@bartgol bartgol added EAMxx PRs focused on capabilities for EAMxx CI: workflow change approved Allow testing on ghci-snl-* machines of PRs that alter a worfklow file workflows labels Feb 13, 2025
@bartgol bartgol requested a review from jgfouca February 13, 2025 03:40
@bartgol bartgol self-assigned this Feb 13, 2025
@jgfouca
Copy link
Member

jgfouca commented Feb 13, 2025

Thanks! I should have tested my last PR, but I didn't know how without merging it.

bartgol added a commit that referenced this pull request Feb 13, 2025
… (PR #7006)

This should fix the eamxx-sa workflow, which has not been running correctly in the nightlies.

Also, run eamxx-sa-testing workflow when the test-all-scream action is updated.

[BFB]
@bartgol bartgol merged commit 050f80d into master Feb 13, 2025
10 checks passed
@bartgol bartgol deleted the bartgol/workflows/fix-test-all-scream-action branch February 13, 2025 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: workflow change approved Allow testing on ghci-snl-* machines of PRs that alter a worfklow file EAMxx PRs focused on capabilities for EAMxx workflows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants