fix: make linter happy

guardian · Jan 8, 2024 · 2899297 · 2899297
1 parent e0d01d0
commit 2899297
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -30303,7 +30303,9 @@ exports.shouldMergePullRequest = void 0;
 const core_1 = __nccwpck_require__(2186);
 const shouldMergePullRequest = ({ pullRequest, config, }) => {
     (0, core_1.info)('Checking pull request is valid');
-    if (!pullRequest.user ||
+    if (
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- keeping code as it was before upgrading eslint
+    !pullRequest.user ||
         pullRequest.user.login !== config.pullRequestAuthor) {
         (0, core_1.info)(`Pull request is not authored by ${config.pullRequestAuthor}, ignoring.`);
         return false;
@@ -30449,7 +30451,6 @@ const parseAdditionalChanges = (additionalChanges) => {
     }
     let json;
     try {
-        // eslint-disable-next-line prefer-const -- this is setting the value above so I don't know what eslint is complaining about
         json = JSON.parse(additionalChanges);
     }
     catch (err) {

diff --git a/src/actions/should-merge-pull-request.ts b/src/actions/should-merge-pull-request.ts
@@ -17,6 +17,7 @@ export const shouldMergePullRequest = ({
 	info('Checking pull request is valid');
 
 	if (
+		// eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- keeping code as it was before upgrading eslint
 		!pullRequest.user ||
 		pullRequest.user.login !== config.pullRequestAuthor
 	) {

diff --git a/src/config.ts b/src/config.ts
@@ -75,7 +75,6 @@ const parseAdditionalChanges = (additionalChanges: string): FileChanges => {
 
 	let json;
 	try {
-		// eslint-disable-next-line prefer-const -- this is setting the value above so I don't know what eslint is complaining about
 		json = JSON.parse(additionalChanges) as FileChanges;
 	} catch (err) {
 		throw new Error('Invalid JSON provided for additional-changes input');