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

fix: remove run-scripted-input-tests-canary job #273

Merged
merged 1 commit into from
May 17, 2024

Conversation

artemrys
Copy link
Member

run-scripted-input-tests-canary job is basically just a part of run-scripted-input-tests-full-matrix which does not take that much time to run in CI so there is no need to keep another job in the reusable workflow.

@artemrys artemrys requested a review from a team as a code owner May 15, 2024 11:41
@mbruzda-splunk
Copy link
Contributor

@artemrys
Copy link
Member Author

afaik full matrix only runs on PRs towards main: Ref: https://github.com/splunk/addonfactory-workflow-addon-release/blob/develop/.github/workflows/reusable-build-test-release.yml#L1919

That's okay, Unix and Linux TA devs can control full-matrix execution using labels.

@artemrys artemrys merged commit 911806f into develop May 17, 2024
6 checks passed
@artemrys artemrys deleted the remove-canary-scripted-inputs branch May 17, 2024 12:54
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2024
@srv-rr-github-token
Copy link

🎉 This PR is included in version 4.16.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants