Skip to content

chore(ci): Ignore PR labeler errors#11494

Merged
kodiakhq[bot] merged 2 commits into
mainfrom
erezrokah-patch-1
Jun 13, 2023
Merged

chore(ci): Ignore PR labeler errors#11494
kodiakhq[bot] merged 2 commits into
mainfrom
erezrokah-patch-1

Conversation

@erezrokah

@erezrokah erezrokah commented Jun 13, 2023

Copy link
Copy Markdown
Member

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

@cq-bot cq-bot added the ci label Jun 13, 2023
@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Jun 13, 2023
@kodiakhq kodiakhq Bot merged commit 8fa1ef6 into main Jun 13, 2023
@kodiakhq kodiakhq Bot deleted the erezrokah-patch-1 branch June 13, 2023 12:38
erezrokah added a commit that referenced this pull request Jun 13, 2023
<!-- 🎉 Thank you for making CloudQuery awesome by submitting a PR 🎉 -->

#### Summary

Woops 🤦 follow up to #11494

<!--
Use the following steps to ensure your PR is ready to be reviewed

- [ ] Read the [contribution guidelines](../blob/main/CONTRIBUTING.md)
🧑‍🎓
- [ ] Run `make lint` to ensure the proposed changes follow the coding
style 🚨 (install golangci-lint
[here](https://golangci-lint.run/usage/install/#local-installation))
- [ ] Run `make test` to ensure the proposed changes pass the tests 🧪
- [ ] If changing a source plugin run `make gen` to ensure docs are up
to date 📝
- [ ] Ensure the status checks below are successful ✅
--->
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

None yet

Development

Successfully merging this pull request may close these issues.

3 participants