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

[bot] Allow for non-PR events in docpaths #333

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Feb 18, 2025

The workflow currently fails if the event is not a pull request, since it attempts to list files for a PR with number 0 and receives a 404 from the GitHub API. Edit the workflow so it does not return an error if the event is not a PR.

The workflow currently fails if the event is not a pull request, since
it attempts to list files for a PR with number 0 and receives a 404 from
the GitHub API. Edit the workflow so it does not return an error if the
event is not a PR.
@ptgott ptgott requested a review from a team as a code owner February 18, 2025 18:39
@ptgott
Copy link
Contributor Author

ptgott commented Feb 18, 2025

Example of a run that fails because of a 404 due to a non-PR event: https://github.com/gravitational/teleport/actions/runs/13394644883/job/37410507330

@hugoShaka hugoShaka merged commit 06995f2 into main Feb 24, 2025
8 checks passed
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.

5 participants