jank/issue-label-manager-action
1
0
Fork 0
forked from actions/issue-label-manager-action
Code Pull requests Activity

moving label toLowercase code up to top of function

Browse source
This commit is contained in:
Benjamin Lannon 2022-03-05 10:54:46 -05:00
commit 45dd92ff81
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
index.js
View file

@ -82,15 +82,15 @@ function diffLabels(oldLabels, newLabels) {
// each entry has two values
// { type: 'create' | 'update' | 'delete', label }
let oldLabelsNames = oldLabels.map((label) => label.name);
let newLabelsNames = newLabels.map((label) => label.name);
let oldLabelsNames = oldLabels.map((label) => label.name.toLowerCase());
let newLabelsNames = newLabels.map((label) => label.name.toLowerCase());
let labelModList = [];
oldLabelsNames.forEach((oLabel) => {
// when using `includes` with strings, the match is case-sensitive
// so we first lowercase both strings when comparing
if (newLabelsNames.toLowerCase().includes(oLabel.toLowerCase())) {
if (newLabelsNames.includes(oLabel)) {
const oldLabel = oldLabels.filter((l) => l.name === oLabel)[0];
const newLabel = newLabels.filter((l) => l.name === oLabel)[0];