Skip to content

Commit

Permalink
fix: accommodate case changes in README header
Browse files Browse the repository at this point in the history
Because of changes to the README, node-core-utils `git node land` is now
broken. This fixes it. Please fast-track?
  • Loading branch information
Trott authored and priyank-p committed Feb 8, 2021
1 parent a10942b commit e8ef932
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
8 changes: 4 additions & 4 deletions lib/collaborators.js
Expand Up @@ -74,10 +74,10 @@ function parseCollaborators(readme, cli) {
const collaborators = new Map();
let m;

const tscIndex = readme.indexOf(TSC_TITLE);
const tsceIndex = readme.indexOf(TSCE_TITLE);
const clIndex = readme.indexOf(CL_TITLE);
const cleIndex = readme.indexOf(CLE_TITLE);
const tscIndex = readme.toUpperCase().indexOf(TSC_TITLE.toUpperCase());
const tsceIndex = readme.toUpperCase().indexOf(TSCE_TITLE.toUpperCase());
const clIndex = readme.toUpperCase().indexOf(CL_TITLE.toUpperCase());
const cleIndex = readme.toUpperCase().indexOf(CLE_TITLE.toUpperCase());

if (tscIndex === -1) {
throw new Error(`Couldn't find ${TSC_TITLE} in the README`);
Expand Down
4 changes: 2 additions & 2 deletions test/fixtures/README/README.md
Expand Up @@ -237,7 +237,7 @@ For more information about the governance of the Node.js project, see
* [bar](https://github.com/bar) -
**Bar User** <bar@example.com> (she/her)

### TSC Emeriti
### TSC emeriti

* [test](https://github.com/test) -
**Test** <test@example.com>
Expand All @@ -257,7 +257,7 @@ For more information about the governance of the Node.js project, see
* [ExtraSpace](https://github.com/extraspace) -
**Extra Space** <extraspace@example.com> (he/him)

### Collaborator Emeriti
### Collaborator emeriti

* [bee](https://github.com/bee) -
**bee** <bee@example.com>
Expand Down

0 comments on commit e8ef932

Please sign in to comment.