updated ignore rules so that dependabot be ignored (BirthdayResearch#73)

* updated ignore rules so that dependabot will not be ignored * updated build and test
DeFiCh · Apr 18, 2021 · 100ad3b · 100ad3b
1 parent 7eb256b
commit 100ad3b
Show file tree

Hide file tree

Showing 4 changed files with 38 additions and 19 deletions.
diff --git a/__tests__/rules/ignore.test.ts b/__tests__/rules/ignore.test.ts
@@ -139,9 +139,9 @@ describe('pull_request', () => {
  await expectIgnore(false)
  })
 
- it('should not ignore opened if Bot', async () => {
+ it('should ignore opened if Bot', async () => {
  set('pull_request', 'opened', 'Bot')
- await expectIgnore(false)
+ await expectIgnore(true)
  })
 
  it('should ignore locked', async () => {
@@ -264,9 +264,9 @@ describe('pull_request_target', () => {
  await expectIgnore(false)
  })
 
- it('should not ignore opened if Bot', async () => {
+ it('should ignore opened if Bot', async () => {
  set('pull_request_target', 'opened', 'Bot')
- await expectIgnore(false)
+ await expectIgnore(true)
  })
 
  it('should ignore locked', async () => {
@@ -378,9 +378,9 @@ describe('issues', () => {
  await expectIgnore(false)
  })
 
- it('should not ignore opened if Bot', async () => {
+ it('should ignore opened if Bot', async () => {
  set('issues', 'opened', 'Bot')
- await expectIgnore(false)
+ await expectIgnore(true)
  })
 
  it('should ignore assigned', async () => {

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/rules/ignore.ts b/src/rules/ignore.ts
@@ -87,11 +87,11 @@ function ignoreClosed(): boolean {
  * To prevent mistakes, this will ignore invalid workflow trigger
  */
 export default async function (): Promise<boolean> {
- if (ignoreLabeledRaceCondition()) {
+ if (ignoreClosed()) {
  return true
  }
 
- if (ignoreClosed()) {
+ if (ignoreLabeledRaceCondition()) {
  return true
  }
 
@@ -103,17 +103,27 @@ export default async function (): Promise<boolean> {
  return ignoreBot()
  }
 
- if (is('pull_request', ['synchronize', 'opened', 'labeled', 'unlabeled'])) {
+ if (is('pull_request', ['synchronize', 'opened'])) {
+ return ignoreBot()
+ }
+
+ if (is('pull_request', ['labeled', 'unlabeled'])) {
  return false
  }
 
- if (
- is('pull_request_target', ['synchronize', 'opened', 'labeled', 'unlabeled'])
- ) {
+ if (is('pull_request_target', ['synchronize', 'opened'])) {
+ return ignoreBot()
+ }
+
+ if (is('pull_request_target', ['labeled', 'unlabeled'])) {
  return false
  }
 
- if (is('issues', ['opened', 'labeled', 'unlabeled'])) {
+ if (is('issues', ['opened'])) {
+ return ignoreBot()
+ }
+
+ if (is('issues', ['labeled', 'unlabeled'])) {
  return false
  }