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

chore(ci): Ignore PR labeler errors #8816

Merged

Conversation

erezrokah
Copy link
Contributor

@erezrokah erezrokah commented Mar 8, 2023

Summary

GitHub does not support more than 100 labels per PR, so the PR labeler action fails when that happens. There's ongoing work to fix it, see actions/labeler#497, but I think we can ignore the action failures for now.

@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Mar 8, 2023
@kodiakhq kodiakhq bot merged commit 0a4717a into cloudquery:main Mar 8, 2023
kodiakhq bot pushed a commit that referenced this pull request Jun 13, 2023

#### Summary

Follow up to #8816. Apparently setting `continue-on-error: true` at the job level doesn't work as expected and still marks the PR as failed. See actions/runner#2347.
Hopefully setting it at the step level works better.

> This should fix an issue on PRs that have more than 100 labels to show as failing, see #11473 
![image](https://github.com/cloudquery/cloudquery/assets/26760571/c6d819c3-ac92-4c3f-b261-4ed8f1eeaf91)


<!--
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Automatically merge once required checks pass
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants