Skip to content

Commit 312efe2

Browse files
committed
v3.2.1: Make include and exclude options case insensitive
1 parent 0823109 commit 312efe2

5 files changed

Lines changed: 26 additions & 4 deletions

File tree

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
# Changelog
22
All notable changes to this project will be documented in this file.
33

4+
## [3.2.1] - 2025-01-19
5+
### Fixed
6+
- Makes the `include` and `exclude` options case insensitive (solves #101)
7+
48
## [3.2.0] - 2024-12-28
59
### Added
610
- The option `include` to include only specific users in the stats.

dist/index.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43372,9 +43372,11 @@ module.exports = (filterStr) => {
4337243372
/***/ 756:
4337343373
/***/ ((module) => {
4337443374

43375+
const sanitize = (str) => String(str).toLowerCase();
43376+
4337543377
module.exports = (filter, username) => {
4337643378
if (filter.test) return filter.test(username);
43377-
if (filter.includes) return filter.includes(username);
43379+
if (filter.includes) return filter.includes(sanitize(username));
4337843380
return false;
4337943381
};
4338043382

@@ -49638,7 +49640,7 @@ module.exports = /*#__PURE__*/JSON.parse('{"name":"mixpanel","description":"A si
4963849640
/***/ ((module) => {
4963949641

4964049642
"use strict";
49641-
module.exports = /*#__PURE__*/JSON.parse('{"name":"pull-request-stats","version":"3.1.0","description":"Github action to print relevant stats about Pull Request reviewers","main":"dist/index.js","type":"commonjs","scripts":{"build":"eslint src && ncc build src/index.js -o dist -a","test":"jest","lint":"eslint ./"},"keywords":[],"author":"Manuel de la Torre","license":"MIT","jest":{"testEnvironment":"node","testMatch":["**/?(*.)+(spec|test).[jt]s?(x)"]},"dependencies":{"@actions/core":"^1.11.1","@actions/github":"^6.0.0","axios":"^1.7.9","humanize-duration":"^3.32.1","i18n-js":"^3.9.2","jsurl":"^0.1.5","lodash.get":"^4.4.2","markdown-table":"^2.0.0","mixpanel":"^0.18.0"},"devDependencies":{"@eslint/eslintrc":"^3.2.0","@eslint/js":"^9.16.0","@vercel/ncc":"^0.38.3","eslint":"^9.16.0","eslint-config-airbnb-base":"^15.0.0","eslint-plugin-import":"^2.31.0","eslint-plugin-jest":"^28.9.0","globals":"^15.13.0","jest":"^29.7.0"},"funding":"https://github.com/sponsors/manuelmhtr","packageManager":"yarn@4.1.0"}');
49643+
module.exports = /*#__PURE__*/JSON.parse('{"name":"pull-request-stats","version":"3.2.1","description":"Github action to print relevant stats about Pull Request reviewers","main":"dist/index.js","type":"commonjs","scripts":{"build":"eslint src && ncc build src/index.js -o dist -a","test":"jest","lint":"eslint ./"},"keywords":[],"author":"Manuel de la Torre","license":"MIT","jest":{"testEnvironment":"node","testMatch":["**/?(*.)+(spec|test).[jt]s?(x)"]},"dependencies":{"@actions/core":"^1.11.1","@actions/github":"^6.0.0","axios":"^1.7.9","humanize-duration":"^3.32.1","i18n-js":"^3.9.2","jsurl":"^0.1.5","lodash.get":"^4.4.2","markdown-table":"^2.0.0","mixpanel":"^0.18.0"},"devDependencies":{"@eslint/eslintrc":"^3.2.0","@eslint/js":"^9.16.0","@vercel/ncc":"^0.38.3","eslint":"^9.16.0","eslint-config-airbnb-base":"^15.0.0","eslint-plugin-import":"^2.31.0","eslint-plugin-jest":"^28.9.0","globals":"^15.13.0","jest":"^29.7.0"},"funding":"https://github.com/sponsors/manuelmhtr","packageManager":"yarn@4.1.0"}');
4964249644

4964349645
/***/ }),
4964449646

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "pull-request-stats",
3-
"version": "3.2.0",
3+
"version": "3.2.1",
44
"description": "Github action to print relevant stats about Pull Request reviewers",
55
"main": "dist/index.js",
66
"type": "commonjs",

src/interactors/getUsers/__tests__/index.test.js

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,4 +28,18 @@ describe('Interactors | getUsers', () => {
2828
const result = getUsers([emptyInput]);
2929
expect(result.length).toEqual(0);
3030
});
31+
32+
it('excludes users even if they have uppercase letters', () => {
33+
const author = { id: '1', login: 'UPPERCASE' };
34+
const customInput = { author, reviews: [] };
35+
const result = getUsers([customInput], { excludeStr: 'uppercase' });
36+
expect(result.length).toEqual(0);
37+
});
38+
39+
it('includes users even if they have uppercase letters', () => {
40+
const author = { id: '1', login: 'UPPERCASE' };
41+
const customInput = { author, reviews: [] };
42+
const result = getUsers([customInput], { includeStr: 'uppercase' });
43+
expect(result.length).toEqual(1);
44+
});
3145
});
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
1+
const sanitize = (str) => String(str).toLowerCase();
2+
13
module.exports = (filter, username) => {
24
if (filter.test) return filter.test(username);
3-
if (filter.includes) return filter.includes(username);
5+
if (filter.includes) return filter.includes(sanitize(username));
46
return false;
57
};

0 commit comments

Comments
 (0)