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

Update workflows for all pull request tests in test.yml #543

Merged
merged 4 commits into from
Jan 4, 2025

Conversation

athackst
Copy link
Owner

@athackst athackst commented Jan 3, 2025

Since forked PRs can't use secrets, this change moves just a build test for the dockerfiles that are changed to a read-only test flow and the build + push docker action is then updated to be used only when merging to main or on a schedule.

@athackst athackst force-pushed the dev/refactor-github-workflows branch 12 times, most recently from cb8ca56 to 8c6d4e3 Compare January 4, 2025 01:10
Since forked PRs can't use secrets, this change moves just a build test for the dockerfiles that are changed to a read-only test flow and the build + push docker action is then updated to be used only when merging to main or on a schedule.
@athackst athackst force-pushed the dev/refactor-github-workflows branch from 8c6d4e3 to 2845e12 Compare January 4, 2025 01:12
@athackst athackst force-pushed the dev/refactor-github-workflows branch 2 times, most recently from 0c576a0 to 50f7bb5 Compare January 4, 2025 01:31
@athackst athackst force-pushed the dev/refactor-github-workflows branch from 50f7bb5 to a7b46cf Compare January 4, 2025 01:39
@athackst athackst merged commit 6522998 into main Jan 4, 2025
54 checks passed
@athackst athackst deleted the dev/refactor-github-workflows branch January 4, 2025 01:46
athackst added a commit that referenced this pull request Jan 4, 2025
@athackst athackst restored the dev/refactor-github-workflows branch January 4, 2025 01:54
athackst added a commit that referenced this pull request Jan 4, 2025
athackst added a commit that referenced this pull request Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant