chore(deps-dev): bump @ridedott/eslint-config from 2.13.22 to 2.19.21

.eslintrc.yaml

@@ -10,7 +10,8 @@ overrides:
     rules:
       '@typescript-eslint/camelcase': 'off'
       '@typescript-eslint/no-magic-numbers': 'off'
-      'max-lines': 'off'
+      jest/require-hook: 'error'
+      max-lines: 'off'
       max-lines-per-function: 'off'
       no-magic-numbers: 'off'
       max-statements: 'off'
@@ -23,4 +24,5 @@ parserOptions:
   project: tsconfig.json
   sourceType: module
 rules:
+  jest/require-hook: 'off'
   no-negated-condition: 'off'

package.json

@@ -13,7 +13,7 @@
   "devDependencies": {
     "@commitlint/cli": "^16.2.3",
     "@commitlint/config-conventional": "^16.2.1",
-    "@ridedott/eslint-config": "^2.13.22",
+    "@ridedott/eslint-config": "^2.19.21",
     "@semantic-release/changelog": "^5.0.1",
     "@semantic-release/exec": "^5.0.0",
     "@semantic-release/git": "^9.0.1",

src/common/getPullRequestCommits.ts

@@ -1,5 +1,5 @@
 import { getOctokit } from '@actions/github';
-import { GraphQlQueryResponseData } from '@octokit/graphql';
+import type { GraphQlQueryResponseData } from '@octokit/graphql';
 
 import { PullRequestCommitNode } from '../types';
 import { IterableList, makeGraphqlIterator } from './makeGraphqlIterator';

src/common/getPullRequestInformation.spec.ts

@@ -159,6 +159,7 @@ describe('getPullRequestInformation', (): void => {
 
       nock('https://api.github.com', {
         reqheaders: {
+          // eslint-disable-next-line jest/no-conditional-in-test
           accept: githubPreviewApiEnabled
             ? 'application/vnd.github.merge-info-preview+json'
             : 'application/vnd.github.v3+json',

src/common/listBranchProtectionRules.ts

@@ -1,5 +1,5 @@
 import { getOctokit } from '@actions/github';
-import { GraphQlQueryResponseData } from '@octokit/graphql';
+import type { GraphQlQueryResponseData } from '@octokit/graphql';
 
 import { IterableList, makeGraphqlIterator } from './makeGraphqlIterator';
 
@@ -52,6 +52,7 @@ export const listBranchProtectionRules = async (
   const branchProtectionRules: BranchProtectionRule[] = [];
 
   for await (const node of iterator) {
+    // eslint-disable-next-line functional/immutable-data
     branchProtectionRules.push(node);
   }
 

src/common/makeGraphqlIterator.ts

@@ -1,9 +1,8 @@
-/* eslint-disable immutable/no-let */
 /* eslint-disable no-await-in-loop */
 /* eslint-disable prefer-destructuring */
 
 import { getOctokit } from '@actions/github';
-import { GraphQlQueryResponseData } from '@octokit/graphql';
+import type { GraphQlQueryResponseData } from '@octokit/graphql';
 
 const MAX_PAGE_SIZE = 100;
 

src/common/merge.spec.ts

@@ -599,6 +599,7 @@ describe('merge', (): void => {
   it('does not log any warnings if last commit was not created by the selected GITHUB_LOGIN and ENABLED_FOR_MANUAL_CHANGES is set to "true"', async (): Promise<void> => {
     expect.assertions(1);
 
+    /* eslint-disable jest/no-conditional-in-test */
     getInputSpy.mockImplementation((name: string): string => {
       if (name === 'ENABLED_FOR_MANUAL_CHANGES') {
         return 'true';
@@ -618,6 +619,7 @@ describe('merge', (): void => {
 
       return '';
     });
+    /* eslint-enable jest/no-conditional-in-test */
 
     const pullRequestInformation: PullRequestInformation = {
       authorLogin: 'dependabot',

src/eventHandlers/continuousIntegrationEnd/index.spec.ts

@@ -138,11 +138,11 @@ describe('continuous integration end event handler', (): void => {
 
     const { check_suite: checkSuite, eventName } = context.payload;
 
-    /* eslint-disable immutable/no-mutation */
+    /* eslint-disable functional/immutable-data */
     context.eventName = 'workflow_run';
     context.payload.workflow_run = checkSuite;
     delete context.payload.check_suite;
-    /* eslint-enable immutable/no-mutation */
+    /* eslint-enable functional/immutable-data */
 
     nock('https://api.github.com')
       .post('/graphql')
@@ -153,12 +153,12 @@ describe('continuous integration end event handler', (): void => {
     await continuousIntegrationEndHandle(octokit, DEPENDABOT_GITHUB_LOGIN, 3);
 
     /* eslint-disable require-atomic-updates */
-    /* eslint-disable immutable/no-mutation */
+    /* eslint-disable functional/immutable-data */
     context.eventName = eventName;
     context.payload.check_suite = checkSuite;
     delete context.payload.workflow_run;
     /* eslint-enable require-atomic-updates */
-    /* eslint-enable immutable/no-mutation */
+    /* eslint-enable functional/immutable-data */
 
     expect(warningSpy).toHaveBeenCalledWith(
       'Unable to fetch pull request information.',

src/eventHandlers/continuousIntegrationEnd/index.ts

@@ -90,26 +90,21 @@ export const continuousIntegrationEndHandle = async (
       pullRequests.map(({ base }: PullRequest): string => base.ref),
     );
 
-  const pullRequestsInformationPromises: Array<
-    Promise<PullRequestInformation | undefined>
-  > = [];
-
-  for (const pullRequest of pullRequests) {
-    pullRequestsInformationPromises.push(
-      getMergeablePullRequestInformationWithRetry(
-        octokit,
-        {
-          pullRequestNumber: pullRequest.number,
-          repositoryName: context.repo.repo,
-          repositoryOwner: context.repo.owner,
-        },
-        { maximum: maximumRetries },
-      ).catch((): undefined => undefined),
-    );
-  }
-
   const pullRequestsInformation = await Promise.all(
-    pullRequestsInformationPromises,
+    pullRequests.map(
+      async (
+        pullRequest: PullRequest,
+      ): Promise<PullRequestInformation | undefined> =>
+        getMergeablePullRequestInformationWithRetry(
+          octokit,
+          {
+            pullRequestNumber: pullRequest.number,
+            repositoryName: context.repo.repo,
+            repositoryOwner: context.repo.owner,
+          },
+          { maximum: maximumRetries },
+        ).catch((): undefined => undefined),
+    ),
   );
 
   const mergePromises: Array<Promise<void>> = [];
@@ -127,6 +122,7 @@ export const continuousIntegrationEndHandle = async (
         )}.`,
       );
 
+      // eslint-disable-next-line functional/immutable-data
       mergePromises.push(
         tryMerge(
           octokit,

src/eventHandlers/pullRequest/index.spec.ts

@@ -106,15 +106,16 @@ describe('pull request event handler', (): void => {
     expect.assertions(0);
 
     const { pull_request: pullRequest } = context.payload;
+    // eslint-disable-next-line functional/immutable-data
     delete context.payload.pull_request;
 
     await pullRequestHandle(octokit, DEPENDABOT_GITHUB_LOGIN, 2);
 
     /* eslint-disable require-atomic-updates */
-    /* eslint-disable immutable/no-mutation */
+    /* eslint-disable functional/immutable-data */
     context.payload.pull_request = pullRequest;
     /* eslint-enable require-atomic-updates */
-    /* eslint-enable immutable/no-mutation */
+    /* eslint-enable functional/immutable-data */
   });
 
   it('logs a warning when it cannot find pull request ID by pull request number (null)', async (): Promise<void> => {

src/index.ts

@@ -37,6 +37,7 @@ const main = async (): Promise<void> => {
   }
 };
 
+// eslint-disable-next-line unicorn/prefer-top-level-await
 main().catch((error: Error): void => {
   setFailed(
     `An unexpected error occurred: ${error.message}, ${

src/utilities/inputParsers.spec.ts

@@ -21,15 +21,15 @@ describe('parseInputMergeMethod', (): void => {
 
     getInputSpy.mockReturnValueOnce('OTHER');
 
-    expect<unknown>(parseInputMergeMethod()).toStrictEqual('SQUASH');
+    expect<unknown>(parseInputMergeMethod()).toBe('SQUASH');
   });
 
   it('returns undefined if merge method is not provided', (): void => {
     expect.assertions(1);
 
     getInputSpy.mockReturnValueOnce('');
 
-    expect<unknown>(parseInputMergeMethod()).toStrictEqual('SQUASH');
+    expect<unknown>(parseInputMergeMethod()).toBe('SQUASH');
   });
 });
 

src/utilities/log.spec.ts

@@ -7,12 +7,13 @@ const errorSpy = jest.spyOn(core, 'error').mockImplementation();
 const infoSpy = jest.spyOn(core, 'info').mockImplementation();
 const warningSpy = jest.spyOn(core, 'warning').mockImplementation();
 
+/* eslint-disable functional/immutable-data */
 const errorWithoutStack = new Error('I am an error.');
 delete errorWithoutStack.stack;
 
 const errorWithStack = new Error('I am an error.');
-/* eslint-disable-next-line immutable/no-mutation */
 errorWithStack.stack = 'I am a stack.';
+/* eslint-enable functional/immutable-data */
 
 describe.each<
   [

src/utilities/prTitleParsers.spec.ts

@@ -12,7 +12,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
 
     parseInputMergePresetSpy.mockReturnValueOnce(undefined);
 
-    expect(checkPullRequestTitleForMergePreset('')).toStrictEqual(true);
+    expect(checkPullRequestTitleForMergePreset('')).toBe(true);
   });
 
   describe('given containing major bump', (): void => {
@@ -25,7 +25,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
 
         parseInputMergePresetSpy.mockReturnValueOnce(mergeCategory);
 
-        expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(false);
+        expect(checkPullRequestTitleForMergePreset(title)).toBe(false);
       },
     );
   });
@@ -40,7 +40,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
 
         parseInputMergePresetSpy.mockReturnValueOnce(mergeCategory);
 
-        expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(false);
+        expect(checkPullRequestTitleForMergePreset(title)).toBe(false);
       },
     );
   });
@@ -55,7 +55,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
         inputParsers.AllowedMergePresets.DEPENDABOT_MINOR,
       );
 
-      expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(true);
+      expect(checkPullRequestTitleForMergePreset(title)).toBe(true);
     });
 
     it('returns false for DEPENDABOT_PATCH', (): void => {
@@ -65,7 +65,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
         inputParsers.AllowedMergePresets.DEPENDABOT_PATCH,
       );
 
-      expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(false);
+      expect(checkPullRequestTitleForMergePreset(title)).toBe(false);
     });
   });
 
@@ -79,7 +79,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
         inputParsers.AllowedMergePresets.DEPENDABOT_MINOR,
       );
 
-      expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(true);
+      expect(checkPullRequestTitleForMergePreset(title)).toBe(true);
     });
 
     it('returns false for DEPENDABOT_PATCH', (): void => {
@@ -89,7 +89,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
         inputParsers.AllowedMergePresets.DEPENDABOT_PATCH,
       );
 
-      expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(false);
+      expect(checkPullRequestTitleForMergePreset(title)).toBe(false);
     });
   });
 
@@ -103,7 +103,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
 
         parseInputMergePresetSpy.mockReturnValueOnce(mergeCategory);
 
-        expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(true);
+        expect(checkPullRequestTitleForMergePreset(title)).toBe(true);
       },
     );
   });
@@ -118,7 +118,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
 
         parseInputMergePresetSpy.mockReturnValueOnce(mergeCategory);
 
-        expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(true);
+        expect(checkPullRequestTitleForMergePreset(title)).toBe(true);
       },
     );
   });
@@ -133,7 +133,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
 
         parseInputMergePresetSpy.mockReturnValueOnce(mergeCategory);
 
-        expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(true);
+        expect(checkPullRequestTitleForMergePreset(title)).toBe(true);
       },
     );
   });
@@ -148,7 +148,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
 
         parseInputMergePresetSpy.mockReturnValueOnce(mergeCategory);
 
-        expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(true);
+        expect(checkPullRequestTitleForMergePreset(title)).toBe(true);
       },
     );
   });
@@ -163,7 +163,7 @@ describe('checkPullRequestTitleForMergePreset', (): void => {
 
         parseInputMergePresetSpy.mockReturnValueOnce(mergeCategory);
 
-        expect(checkPullRequestTitleForMergePreset(title)).toStrictEqual(false);
+        expect(checkPullRequestTitleForMergePreset(title)).toBe(false);
       },
     );
   });