Skip to content

Commit a6da04e

Browse files
committed
refactor(rules): refactor indentation and readability for case rules
1 parent e630d86 commit a6da04e

File tree

4 files changed

+24
-18
lines changed

4 files changed

+24
-18
lines changed

@commitlint/rules/src/header-case.ts

+7-5
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
1-
import * as ensure from '@commitlint/ensure';
1+
import {TargetCaseType, case as ensureCase} from '@commitlint/ensure';
22
import message from '@commitlint/message';
33
import {Rule} from './types';
44

55
const negated = (when?: string) => when === 'never';
66

7-
export const headerCase: Rule<
8-
ensure.TargetCaseType | ensure.TargetCaseType[]
9-
> = (parsed, when = 'always', value = []) => {
7+
export const headerCase: Rule<TargetCaseType | TargetCaseType[]> = (
8+
parsed,
9+
when = 'always',
10+
value = []
11+
) => {
1012
const {header} = parsed;
1113

1214
if (typeof header !== 'string' || !header.match(/^[a-z]/i)) {
@@ -24,7 +26,7 @@ export const headerCase: Rule<
2426
});
2527

2628
const result = checks.some(check => {
27-
const r = ensure.case(header, check.case);
29+
const r = ensureCase(header, check.case);
2830
return negated(check.when) ? !r : r;
2931
});
3032

@commitlint/rules/src/scope-case.ts

+7-5
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
1-
import * as ensure from '@commitlint/ensure';
1+
import {TargetCaseType, case as ensureCase} from '@commitlint/ensure';
22
import message from '@commitlint/message';
33
import {Rule} from './types';
44

55
const negated = (when?: string) => when === 'never';
66

7-
export const scopeCase: Rule<
8-
ensure.TargetCaseType | ensure.TargetCaseType[]
9-
> = (parsed, when = 'always', value = []) => {
7+
export const scopeCase: Rule<TargetCaseType | TargetCaseType[]> = (
8+
parsed,
9+
when = 'always',
10+
value = []
11+
) => {
1012
const {scope} = parsed;
1113

1214
if (!scope) {
@@ -30,7 +32,7 @@ export const scopeCase: Rule<
3032

3133
const result = checks.some(check => {
3234
const r = scopeSegments.every(
33-
segment => delimiters.test(segment) || ensure.case(segment, check.case)
35+
segment => delimiters.test(segment) || ensureCase(segment, check.case)
3436
);
3537

3638
return negated(check.when) ? !r : r;

@commitlint/rules/src/subject-case.ts

+7-5
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
1-
import * as ensure from '@commitlint/ensure';
1+
import {TargetCaseType, case as ensureCase} from '@commitlint/ensure';
22
import message from '@commitlint/message';
33
import {Rule} from './types';
44

55
const negated = (when?: string) => when === 'never';
66

7-
export const subjectCase: Rule<
8-
ensure.TargetCaseType | ensure.TargetCaseType[]
9-
> = (parsed, when = 'always', value = []) => {
7+
export const subjectCase: Rule<TargetCaseType | TargetCaseType[]> = (
8+
parsed,
9+
when = 'always',
10+
value = []
11+
) => {
1012
const {subject} = parsed;
1113

1214
if (typeof subject !== 'string' || !subject.match(/^[a-z]/i)) {
@@ -24,7 +26,7 @@ export const subjectCase: Rule<
2426
});
2527

2628
const result = checks.some(check => {
27-
const r = ensure.case(subject, check.case);
29+
const r = ensureCase(subject, check.case);
2830
return negated(check.when) ? !r : r;
2931
});
3032

@commitlint/rules/src/type-case.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
1-
import * as ensure from '@commitlint/ensure';
1+
import {TargetCaseType, case as ensureCase} from '@commitlint/ensure';
22
import message from '@commitlint/message';
33
import {Rule} from './types';
44

55
const negated = (when?: string) => when === 'never';
66

7-
export const typeCase: Rule<ensure.TargetCaseType | ensure.TargetCaseType[]> = (
7+
export const typeCase: Rule<TargetCaseType | TargetCaseType[]> = (
88
parsed,
99
when = 'always',
1010
value = []
@@ -26,7 +26,7 @@ export const typeCase: Rule<ensure.TargetCaseType | ensure.TargetCaseType[]> = (
2626
});
2727

2828
const result = checks.some(check => {
29-
const r = ensure.case(type, check.case);
29+
const r = ensureCase(type, check.case);
3030
return negated(check.when) ? !r : r;
3131
});
3232

0 commit comments

Comments
 (0)