Skip to content

Commit 996757d

Browse files
nitzanashidependabot[bot]
nitzanashi
and
dependabot[bot]
authored
Release 2.1.1 (#38)
* Bump @vercel/ncc from 0.29.0 to 0.31.0 (#17) Bumps [@vercel/ncc](https://github.com/vercel/ncc) from 0.29.0 to 0.31.0. - [Release notes](https://github.com/vercel/ncc/releases) - [Commits](vercel/ncc@0.29.0...0.31.0) --- updated-dependencies: - dependency-name: "@vercel/ncc" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.0.6 to 27.1.1 (#16) Bumps [jest](https://github.com/facebook/jest) from 27.0.6 to 27.1.1. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.0.6...v27.1.1) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.1.1 to 27.2.0 (#18) Bumps [jest](https://github.com/facebook/jest) from 27.1.1 to 27.2.0. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.1.1...v27.2.0) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @vercel/ncc from 0.31.0 to 0.31.1 (#19) Bumps [@vercel/ncc](https://github.com/vercel/ncc) from 0.31.0 to 0.31.1. - [Release notes](https://github.com/vercel/ncc/releases) - [Commits](vercel/ncc@0.31.0...0.31.1) --- updated-dependencies: - dependency-name: "@vercel/ncc" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.2.0 to 27.2.1 (#21) Bumps [jest](https://github.com/facebook/jest) from 27.2.0 to 27.2.1. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.2.0...v27.2.1) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.2.1 to 27.3.0 (#30) Bumps [jest](https://github.com/facebook/jest) from 27.2.1 to 27.3.0. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.2.1...v27.3.0) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @types/fs-extra from 9.0.12 to 9.0.13 (#22) Bumps [@types/fs-extra](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/fs-extra) from 9.0.12 to 9.0.13. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/fs-extra) --- updated-dependencies: - dependency-name: "@types/fs-extra" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump eslint from 7.32.0 to 8.0.1 (#29) Bumps [eslint](https://github.com/eslint/eslint) from 7.32.0 to 8.0.1. - [Release notes](https://github.com/eslint/eslint/releases) - [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md) - [Commits](eslint/eslint@v7.32.0...v8.0.1) --- updated-dependencies: - dependency-name: eslint dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @actions/core from 1.5.0 to 1.6.0 (#24) Bumps [@actions/core](https://github.com/actions/toolkit/tree/HEAD/packages/core) from 1.5.0 to 1.6.0. - [Release notes](https://github.com/actions/toolkit/releases) - [Changelog](https://github.com/actions/toolkit/blob/main/packages/core/RELEASES.md) - [Commits](https://github.com/actions/toolkit/commits/HEAD/packages/core) --- updated-dependencies: - dependency-name: "@actions/core" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.3.0 to 27.3.1 (#31) Bumps [jest](https://github.com/facebook/jest) from 27.3.0 to 27.3.1. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.3.0...v27.3.1) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Handle `listReviews` pagination (#32) * pagination * Limit for testing * Limit for testing * Limit for testing * Test * Test * Test * Test * Test * Test * Test * Test * Test * Test * Test * Test * Test * Changelog and cleanup * Build * Capotal * Release 2.1.0 (#34) * Bump @vercel/ncc from 0.29.0 to 0.31.0 (#17) Bumps [@vercel/ncc](https://github.com/vercel/ncc) from 0.29.0 to 0.31.0. - [Release notes](https://github.com/vercel/ncc/releases) - [Commits](vercel/ncc@0.29.0...0.31.0) --- updated-dependencies: - dependency-name: "@vercel/ncc" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.0.6 to 27.1.1 (#16) Bumps [jest](https://github.com/facebook/jest) from 27.0.6 to 27.1.1. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.0.6...v27.1.1) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.1.1 to 27.2.0 (#18) Bumps [jest](https://github.com/facebook/jest) from 27.1.1 to 27.2.0. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.1.1...v27.2.0) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @vercel/ncc from 0.31.0 to 0.31.1 (#19) Bumps [@vercel/ncc](https://github.com/vercel/ncc) from 0.31.0 to 0.31.1. - [Release notes](https://github.com/vercel/ncc/releases) - [Commits](vercel/ncc@0.31.0...0.31.1) --- updated-dependencies: - dependency-name: "@vercel/ncc" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.2.0 to 27.2.1 (#21) Bumps [jest](https://github.com/facebook/jest) from 27.2.0 to 27.2.1. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.2.0...v27.2.1) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.2.1 to 27.3.0 (#30) Bumps [jest](https://github.com/facebook/jest) from 27.2.1 to 27.3.0. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.2.1...v27.3.0) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @types/fs-extra from 9.0.12 to 9.0.13 (#22) Bumps [@types/fs-extra](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/fs-extra) from 9.0.12 to 9.0.13. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/fs-extra) --- updated-dependencies: - dependency-name: "@types/fs-extra" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump eslint from 7.32.0 to 8.0.1 (#29) Bumps [eslint](https://github.com/eslint/eslint) from 7.32.0 to 8.0.1. - [Release notes](https://github.com/eslint/eslint/releases) - [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md) - [Commits](eslint/eslint@v7.32.0...v8.0.1) --- updated-dependencies: - dependency-name: eslint dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @actions/core from 1.5.0 to 1.6.0 (#24) Bumps [@actions/core](https://github.com/actions/toolkit/tree/HEAD/packages/core) from 1.5.0 to 1.6.0. - [Release notes](https://github.com/actions/toolkit/releases) - [Changelog](https://github.com/actions/toolkit/blob/main/packages/core/RELEASES.md) - [Commits](https://github.com/actions/toolkit/commits/HEAD/packages/core) --- updated-dependencies: - dependency-name: "@actions/core" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jest from 27.3.0 to 27.3.1 (#31) Bumps [jest](https://github.com/facebook/jest) from 27.3.0 to 27.3.1. - [Release notes](https://github.com/facebook/jest/releases) - [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md) - [Commits](jestjs/jest@v27.3.0...v27.3.1) --- updated-dependencies: - dependency-name: jest dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Handle `listReviews` pagination (#32) * pagination * Limit for testing * Limit for testing * Limit for testing * Test * Test * Test * Test * Test * Test * Test * Test * Test * Test * Test * Test * Test * Changelog and cleanup * Build * Capotal * Release Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Last review per user is being evaluated even if it is just a comment (#37) * Filter out unrelevant Reviewers * Tweak * Tweak * Release 2.1.1 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
1 parent d7417d5 commit 996757d

File tree

6 files changed

+19
-4
lines changed

6 files changed

+19
-4
lines changed

‎CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@ All notable changes to this project will be documented in this file.
33
The format is based on [Keep a Changelog](http://keepachangelog.com/)
44
and this project adheres to [Semantic Versioning](http://semver.org/).
55

6+
## 2.1.1 - 05.11.2021
7+
8+
### Fixed
9+
- [#35](https://github.com/zattoo/reviewers/issues/35) Last review per user is being evaluated even if it is just a comment
10+
611
## 2.1.0 - 25.10.2021
712

813
### Fixed

‎dist/index.js

Lines changed: 5 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/index.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package-lock.json

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "reviewers",
3-
"version": "2.1.0",
3+
"version": "2.1.1",
44
"description": "GitHub Action to recognize and assign reviewers and codeowners",
55
"main": "src/index.js",
66
"scripts": {

‎src/index.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,11 @@ const PATH_PREFIX = process.env.GITHUB_WORKSPACE;
216216
const user = review.user.login;
217217
const hasUserAlready = Boolean(latestReviews[user]);
218218

219+
// https://docs.github.com/en/graphql/reference/enums#pullrequestreviewstate
220+
if (!['APPROVED', 'CHANGES_REQUESTED', 'DISMISSED'].includes(review.state)) {
221+
return;
222+
}
223+
219224
if (!hasUserAlready) {
220225
latestReviews[user] = review;
221226
} else if (review.submitted_at > latestReviews[user].submitted_at) {

0 commit comments

Comments
 (0)