Skip to content

Commit ec174b3

Browse files
danezdanez
authored
change prettier parens (#746)
1 parent 2020a5b commit ec174b3

File tree

69 files changed

+236
-229
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

69 files changed

+236
-229
lines changed

‎.prettierrc.cjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict';
22

33
module.exports = {
4-
arrowParens: 'avoid',
4+
arrowParens: 'always',
55
trailingComma: 'all',
66
useTabs: false,
77
printWidth: 80,

‎benchmark/index.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ const preparedSuites = [];
4646
suites.forEach(({ name, files }) => {
4747
const suite = new Benchmark.Suite(name);
4848

49-
files.forEach(file => {
49+
files.forEach((file) => {
5050
const code = fs.readFileSync(path.join(__dirname, file), 'utf-8');
5151
const options = { filename: file, babelrc: false, configFile: false };
5252
const newOptions = { babelOptions: options };

‎packages/react-docgen-cli/src/commands/parse/command.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ program
123123
});
124124
// we use slash to convert windows backslashes to unix format so fast-glob works
125125
const files = await glob(globs.map(slash), {
126-
ignore: finalIgnores?.map(ignorePath => {
126+
ignore: finalIgnores?.map((ignorePath) => {
127127
ignorePath = ignorePath.trim();
128128
// If the ignore glob starts with a dot we need to resolve the path to an
129129
// absolute path in order for it to work
@@ -139,7 +139,7 @@ program
139139
let errorEncountered = false;
140140

141141
await Promise.all(
142-
files.map(async path => {
142+
files.map(async (path) => {
143143
debug(`Reading file ${path}`);
144144
const content = await readFile(path, 'utf-8');
145145

‎packages/react-docgen-cli/src/commands/parse/options/loadOptions.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ export default async function loadOptions(input: {
2323

2424
const handlers = input.handler
2525
? await Promise.all(
26-
input.handler.map(async handler => {
26+
input.handler.map(async (handler) => {
2727
return await loadReactDocgenPlugin<Handler>(
2828
handler,
2929
'handler',

‎packages/react-docgen-cli/tests/integration/cli-test.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ describe('cli', () => {
174174

175175
describe('importer', () => {
176176
describe('accepts the names of builtin importers', () => {
177-
test.each(Object.keys(builtinImporters))('%s', async importer => {
177+
test.each(Object.keys(builtinImporters))('%s', async (importer) => {
178178
await withFixture('basic', async ({ dir, run }) => {
179179
const { stdout, stderr } = await run([
180180
`--importer=${importer}`,
@@ -272,7 +272,7 @@ describe('cli', () => {
272272

273273
describe('handlers', () => {
274274
describe('accepts the names of builtin handlers', () => {
275-
test.each(Object.keys(builtinHandlers))('%s', async importer => {
275+
test.each(Object.keys(builtinHandlers))('%s', async (importer) => {
276276
await withFixture('basic', async ({ dir, run }) => {
277277
const { stdout, stderr } = await run([
278278
`--handler=${importer}`,

‎packages/react-docgen-cli/tests/integration/resolver-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import withFixture from './utils/withFixture';
55

66
describe('resolver', () => {
77
describe('accepts the names of builtin resolver configs', () => {
8-
test.each(Object.values(ResolverConfigs))('%s', async importer => {
8+
test.each(Object.values(ResolverConfigs))('%s', async (importer) => {
99
await withFixture('basic', async ({ dir, run }) => {
1010
const { stdout, stderr } = await run([
1111
`--resolver=${importer}`,

‎packages/react-docgen/src/babelParser.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ function buildPluginList(
5757

5858
// Ensure that the estree plugin is never active
5959
// TODO add test
60-
return plugins.filter(plugin => plugin !== 'estree');
60+
return plugins.filter((plugin) => plugin !== 'estree');
6161
}
6262

6363
function buildParserOptions(options: TransformOptions): ParserOptions {

‎packages/react-docgen/src/handlers/__tests__/codeTypeHandler-test.ts

+7-7
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ describe('codeTypeHandler', () => {
2727
});
2828

2929
const mockImporter = makeMockImporter({
30-
something: stmtLast =>
30+
something: (stmtLast) =>
3131
stmtLast<ExportNamedDeclaration>(`
3232
export type Props = {
3333
foo: string,
@@ -169,7 +169,7 @@ describe('codeTypeHandler', () => {
169169
});
170170

171171
describe('special generic type annotations', () => {
172-
['$ReadOnly', '$Exact'].forEach(annotation => {
172+
['$ReadOnly', '$Exact'].forEach((annotation) => {
173173
test(`unwraps ${annotation}<...>`, () => {
174174
const flowTypesSrc = `
175175
${annotation}<{
@@ -195,7 +195,7 @@ describe('codeTypeHandler', () => {
195195

196196
describe('TypeAlias', () => {
197197
describe('class definition for flow <0.53', () => {
198-
testCodeTypeHandler(propTypesSrc =>
198+
testCodeTypeHandler((propTypesSrc) =>
199199
parse.statement(
200200
template(
201201
'class Foo extends Component<void, Props, void> {}',
@@ -206,15 +206,15 @@ describe('codeTypeHandler', () => {
206206
});
207207

208208
describe('class definition for flow >=0.53 without State', () => {
209-
testCodeTypeHandler(propTypesSrc =>
209+
testCodeTypeHandler((propTypesSrc) =>
210210
parse.statement(
211211
template('class Foo extends Component<Props> {}', propTypesSrc),
212212
),
213213
);
214214
});
215215

216216
describe('class definition for flow >=0.53 with State', () => {
217-
testCodeTypeHandler(propTypesSrc =>
217+
testCodeTypeHandler((propTypesSrc) =>
218218
parse.statement(
219219
template(
220220
'class Foo extends Component<Props, State> {}',
@@ -225,7 +225,7 @@ describe('codeTypeHandler', () => {
225225
});
226226

227227
describe('class definition with inline props', () => {
228-
testCodeTypeHandler(propTypesSrc =>
228+
testCodeTypeHandler((propTypesSrc) =>
229229
parse.statement(
230230
template(
231231
'class Foo extends Component { props: Props; }',
@@ -237,7 +237,7 @@ describe('codeTypeHandler', () => {
237237

238238
describe('stateless component', () => {
239239
testCodeTypeHandler(
240-
propTypesSrc =>
240+
(propTypesSrc) =>
241241
parse
242242
.statement(template('(props: Props) => <div />;', propTypesSrc))
243243
.get('expression') as NodePath<ArrowFunctionExpression>,

‎packages/react-docgen/src/handlers/__tests__/componentDocblockHandler-test.ts

+28-26
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ describe('componentDocblockHandler', () => {
138138
const importDef = useDefault ? `${importName}` : `{ ${importName} }`;
139139

140140
const mockImporter = makeMockImporter({
141-
test1: stmtLast =>
141+
test1: (stmtLast) =>
142142
stmtLast(
143143
`
144144
/**
@@ -150,7 +150,7 @@ describe('componentDocblockHandler', () => {
150150
0,
151151
).get('declaration') as NodePath,
152152

153-
test2: stmtLast =>
153+
test2: (stmtLast) =>
154154
stmtLast<ExportDefaultDeclaration>(`
155155
import ${importDef} from 'test1';
156156
export default ${importName};
@@ -189,7 +189,7 @@ describe('componentDocblockHandler', () => {
189189
describe('React.createClass', () => {
190190
testDockblockHandler(
191191
'var Component = React.createClass({})',
192-
src =>
192+
(src) =>
193193
parse
194194
.statementLast(src)
195195
.get('declarations.0.init.arguments.0') as NodePath<ObjectExpression>,
@@ -198,15 +198,17 @@ describe('componentDocblockHandler', () => {
198198
});
199199

200200
describe('ClassDeclaration', () => {
201-
testDockblockHandler('class Component {}', src => parse.statementLast(src));
202-
testDecorators('class Component {}', src => parse.statementLast(src));
201+
testDockblockHandler('class Component {}', (src) =>
202+
parse.statementLast(src),
203+
);
204+
testDecorators('class Component {}', (src) => parse.statementLast(src));
203205
testImports('export class Component {}', 'Component');
204206
});
205207

206208
describe('ClassExpression', () => {
207209
testDockblockHandler(
208210
'var Component = class {};',
209-
src =>
211+
(src) =>
210212
parse
211213
.statementLast<VariableDeclaration>(src)
212214
.get('declarations.0.init') as NodePath<ClassExpression>,
@@ -215,21 +217,21 @@ describe('componentDocblockHandler', () => {
215217
});
216218

217219
describe('Stateless functions', () => {
218-
testDockblockHandler('function Component() {}', src =>
220+
testDockblockHandler('function Component() {}', (src) =>
219221
parse.statementLast(src),
220222
);
221223
testImports('export function Component() {}', 'Component');
222224
testDockblockHandler(
223225
'var Component = function () {};',
224-
src =>
226+
(src) =>
225227
parse
226228
.statementLast<VariableDeclaration>(src)
227229
.get('declarations.0.init') as NodePath<FunctionExpression>,
228230
);
229231
testImports('export var Component = function () {};', 'Component');
230232
testDockblockHandler(
231233
'var Component = () => {}',
232-
src =>
234+
(src) =>
233235
parse
234236
.statementLast<VariableDeclaration>(src)
235237
.get('declarations.0.init') as NodePath<ArrowFunctionExpression>,
@@ -241,7 +243,7 @@ describe('componentDocblockHandler', () => {
241243
describe('Default React.createClass export', () => {
242244
testDockblockHandler(
243245
'export default React.createClass({});',
244-
src =>
246+
(src) =>
245247
parse
246248
.statementLast(src)
247249
.get('declaration.arguments.0') as NodePath<ObjectExpression>,
@@ -251,14 +253,14 @@ describe('componentDocblockHandler', () => {
251253
describe('Default class declaration export', () => {
252254
testDockblockHandler(
253255
'export default class Component {}',
254-
src =>
256+
(src) =>
255257
parse
256258
.statementLast(src)
257259
.get('declaration') as NodePath<ClassDeclaration>,
258260
);
259261
testDecorators(
260262
'class Component {}',
261-
src =>
263+
(src) =>
262264
parse
263265
.statementLast(src)
264266
.get('declaration') as NodePath<ClassDeclaration>,
@@ -269,14 +271,14 @@ describe('componentDocblockHandler', () => {
269271
describe('Default class expression export', () => {
270272
testDockblockHandler(
271273
'export default class {}',
272-
src =>
274+
(src) =>
273275
parse
274276
.statementLast(src)
275277
.get('declaration') as NodePath<ClassExpression>,
276278
);
277279
testDecorators(
278280
'class {}',
279-
src =>
281+
(src) =>
280282
parse
281283
.statementLast(src)
282284
.get('declaration') as NodePath<ClassExpression>,
@@ -288,7 +290,7 @@ describe('componentDocblockHandler', () => {
288290
describe('named function', () => {
289291
testDockblockHandler(
290292
'export default function Component() {}',
291-
src =>
293+
(src) =>
292294
parse
293295
.statementLast(src)
294296
.get('declaration') as NodePath<FunctionDeclaration>,
@@ -298,7 +300,7 @@ describe('componentDocblockHandler', () => {
298300
describe('anonymous function', () => {
299301
testDockblockHandler(
300302
'export default function() {}',
301-
src =>
303+
(src) =>
302304
parse
303305
.statementLast(src)
304306
.get('declaration') as NodePath<FunctionDeclaration>,
@@ -308,7 +310,7 @@ describe('componentDocblockHandler', () => {
308310
describe('arrow function', () => {
309311
testDockblockHandler(
310312
'export default () => {}',
311-
src =>
313+
(src) =>
312314
parse
313315
.statementLast<ExportDefaultDeclaration>(src)
314316
.get('declaration') as NodePath<ArrowFunctionExpression>,
@@ -321,7 +323,7 @@ describe('componentDocblockHandler', () => {
321323
describe('Named React.createClass export', () => {
322324
testDockblockHandler(
323325
'export var Component = React.createClass({});',
324-
src =>
326+
(src) =>
325327
parse
326328
.statementLast(src)
327329
.get(
@@ -333,14 +335,14 @@ describe('componentDocblockHandler', () => {
333335
describe('Named class declaration export', () => {
334336
testDockblockHandler(
335337
'export class Component {}',
336-
src =>
338+
(src) =>
337339
parse
338340
.statementLast(src)
339341
.get('declaration') as NodePath<ClassDeclaration>,
340342
);
341343
testDecorators(
342344
'class Component {}',
343-
src =>
345+
(src) =>
344346
parse
345347
.statementLast(src)
346348
.get('declaration') as NodePath<ClassDeclaration>,
@@ -352,7 +354,7 @@ describe('componentDocblockHandler', () => {
352354
describe('named function', () => {
353355
testDockblockHandler(
354356
'export function Component() {}',
355-
src =>
357+
(src) =>
356358
parse
357359
.statementLast(src)
358360
.get('declaration') as NodePath<FunctionDeclaration>,
@@ -362,7 +364,7 @@ describe('componentDocblockHandler', () => {
362364
describe('anonymous function', () => {
363365
testDockblockHandler(
364366
'export var Component = function() {}',
365-
src =>
367+
(src) =>
366368
parse
367369
.statementLast(src)
368370
.get('declaration') as NodePath<FunctionExpression>,
@@ -372,7 +374,7 @@ describe('componentDocblockHandler', () => {
372374
describe('arrow function', () => {
373375
testDockblockHandler(
374376
'export var Component = () => {}',
375-
src =>
377+
(src) =>
376378
parse
377379
.statementLast(src)
378380
.get('declaration') as NodePath<ArrowFunctionExpression>,
@@ -389,7 +391,7 @@ describe('componentDocblockHandler', () => {
389391
`
390392
React.forwardRef((props, ref) => {});
391393
import React from "react";`,
392-
src =>
394+
(src) =>
393395
parse
394396
.statement(src, -2)
395397
.get('expression') as NodePath<CallExpression>,
@@ -409,7 +411,7 @@ describe('componentDocblockHandler', () => {
409411
React.memo(React.forwardRef((props, ref) => {}));
410412
import React from "react";
411413
`,
412-
src =>
414+
(src) =>
413415
parse
414416
.statement(src, -2)
415417
.get('expression') as NodePath<CallExpression>,
@@ -432,7 +434,7 @@ describe('componentDocblockHandler', () => {
432434
React.forwardRef(Component);
433435
import React from "react";
434436
`,
435-
src =>
437+
(src) =>
436438
parse
437439
.statement(src, -2)
438440
.get('expression') as NodePath<CallExpression>,

‎packages/react-docgen/src/handlers/__tests__/componentMethodsHandler-test.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -25,19 +25,19 @@ describe('componentMethodsHandler', () => {
2525
});
2626

2727
const mockImporter = makeMockImporter({
28-
baz: stmtLast =>
28+
baz: (stmtLast) =>
2929
stmtLast<ExportDefaultDeclaration>(`
3030
export default (foo: string): string => {};
3131
`).get('declaration'),
3232

33-
foo: stmtLast =>
33+
foo: (stmtLast) =>
3434
stmtLast<ExportDefaultDeclaration>(`
3535
export default function(bar: number): number {
3636
return bar;
3737
}
3838
`).get('declaration'),
3939

40-
doFoo: stmtLast =>
40+
doFoo: (stmtLast) =>
4141
stmtLast<ExportDefaultDeclaration>(`
4242
export default () => {};
4343
`).get('declaration'),

0 commit comments

Comments
 (0)