Skip to content

Commit

Permalink
lint: ignore comments in max-lines
Browse files Browse the repository at this point in the history
  • Loading branch information
ealush committed Jul 15, 2022
1 parent a9070d9 commit 47d688a
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ module.exports = {
excludedFiles: ['./**/__tests__/**/*.*'],
rules: {
'max-depth': [1, { max: 3 }],
'max-lines-per-function': [1, { max: 45 }],
'max-lines-per-function': [1, { max: 45, skipComments: true }],
'max-nested-callbacks': [1, { max: 2 }],
'max-statements': [1, { max: 10 }],
},
Expand Down
1 change: 0 additions & 1 deletion packages/n4s/src/runtime/enforceEager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import { transformResult } from 'transformResult';

type IRules = n4s.IRules<Record<string, any>>;

// eslint-disable-next-line max-lines-per-function
export default function enforceEager(value: RuleValue): IRules {
const target = {} as IRules;

Expand Down
2 changes: 1 addition & 1 deletion packages/vest/src/core/test/test.memo.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import { useCursor } from 'isolateHooks';
import registerPrevRunTest from 'registerPrevRunTest';
import { useSuiteId } from 'stateHooks';
import type { TestBase } from 'test';
// eslint-disable-next-line max-lines-per-function

export default function bindTestMemo(test: TestBase): {
(fieldName: string, test: TestFn, deps: unknown[]): VestTest;
(fieldName: string, message: string, test: TestFn, deps: unknown[]): VestTest;
Expand Down
2 changes: 1 addition & 1 deletion packages/vest/src/exports/parser.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { hasOwnProperty, invariant, isPositive } from 'vest-utils';

import { SuiteSummary } from 'SuiteSummaryTypes';

// eslint-disable-next-line max-lines-per-function, max-statements
// eslint-disable-next-line max-statements
export function parse(summary: SuiteSummary): ParsedVestObject {
invariant(
summary && hasOwnProperty(summary, 'valid'),
Expand Down
2 changes: 1 addition & 1 deletion packages/vest/src/hooks/exclusive.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ skip.group = (item: ExclusionItem) =>

//Checks whether a certain test profile excluded by any of the exclusion groups.

// eslint-disable-next-line complexity, max-statements, max-lines-per-function
// eslint-disable-next-line complexity, max-statements
export function isExcluded(testObject: VestTest): boolean {
const { fieldName, groupName } = testObject;

Expand Down

1 comment on commit 47d688a

@vercel
Copy link

@vercel vercel bot commented on 47d688a Jul 15, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully deployed to the following URLs:

vest-next – ./website

vest-website.vercel.app
vest-next-ealush.vercel.app
vest-next-git-latest-ealush.vercel.app
vest-next.vercel.app

Please sign in to comment.