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

Change Actions tests to required #63

Merged
merged 2 commits into from
May 30, 2024
Merged

Conversation

kroenlein
Copy link
Collaborator

This PR modifies the GitHub Actions versions of the repository checks to required (as opposed to optional). These should now pass as the main archive now contains an initialize action (as opposed to only the branch to be merged).

If this passes and deploys as expected, a third PR in the series will remove Travis configuration from the repository and remove the gate from the repository settings.

Copy link

@anoto-moniz anoto-moniz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keep in mind there's also a repo setting you need to set for GitHub to gate on them. continue-on-error just impacts when they fail.

@kroenlein kroenlein merged commit aadbe41 into master May 30, 2024
3 checks passed
@kroenlein kroenlein deleted the maintain/migrate-to-actions-2 branch May 30, 2024 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants