Skip to content

Commit

Permalink
fix: don't block merge by mergeable_state: 'blocking'
Browse files Browse the repository at this point in the history
Mergeable state is seems to be broken - https://github.com/orgs/community/discussions/73849
Let's not block PRs because of that, we are checking the CI anyway
  • Loading branch information
jamacku committed Nov 8, 2023
1 parent ead8eed commit b20a73f
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 6 deletions.
4 changes: 3 additions & 1 deletion dist/action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/action.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

6 changes: 4 additions & 2 deletions src/action.ts
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,10 @@ async function action(
break;

default:
err.push(
`🔴 Pull Request doesn't meet requirements, \`mergeable_state\` is \`${pr?.mergeableState}\``
// Mergeable state is seems to be broken - https://github.com/orgs/community/discussions/73849
// Let's not block PRs because of that, we are checking the CI anyway
message.push(
`🟠 Pull Request doesn't meet requirements, \`mergeable_state\` is \`${pr?.mergeableState}\``
);
}

Expand Down

0 comments on commit b20a73f

Please sign in to comment.