Skip to content

Commit e83f868

Browse files
SamChou19815meta-codesync[bot]
authored andcommitted
Stop using prettier-plugin-hermes-parser for formatting
Summary: Changelog: [Internal] Differential Revision: D111190630
1 parent 102fde7 commit e83f868

60 files changed

Lines changed: 255 additions & 504 deletions

File tree

Some content is hidden

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

.prettierrc.js

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,6 @@
77
* @format
88
*/
99

10-
let plugins = ['prettier-plugin-hermes-parser'];
11-
try {
12-
plugins = require('./.prettier-plugins.fb.js');
13-
} catch {}
14-
1510
module.exports = {
1611
arrowParens: 'avoid',
1712
bracketSameLine: true,
@@ -20,7 +15,6 @@ module.exports = {
2015
singleQuote: true,
2116
trailingComma: 'all',
2217
endOfLine: 'lf',
23-
plugins,
2418
overrides: [
2519
{
2620
files: ['*.code-workspace'],
@@ -31,7 +25,7 @@ module.exports = {
3125
{
3226
files: ['*.js', '*.js.flow'],
3327
options: {
34-
parser: 'hermes',
28+
parser: 'flow',
3529
},
3630
},
3731
{

flow-typed/environment/node.js

Lines changed: 15 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -457,8 +457,7 @@ declare class child_process$ChildProcessTyped<
457457
TStdin extends stream$Writable | null,
458458
TStdout extends stream$Readable | null,
459459
TStderr extends stream$Readable | null,
460-
> extends events$EventEmitter
461-
{
460+
> extends events$EventEmitter {
462461
+stdin: TStdin;
463462
+stdout: TStdout;
464463
+stderr: TStderr;
@@ -578,8 +577,7 @@ declare module 'child_process' {
578577
declare function execFile(
579578
file: string,
580579
argsOrCallback?:
581-
| ReadonlyArray<string>
582-
| child_process$execFileCallback<string>,
580+
ReadonlyArray<string> | child_process$execFileCallback<string>,
583581
callback?: child_process$execFileCallback<string>,
584582
): child_process$ChildProcessTyped<
585583
stream$Writable,
@@ -928,14 +926,7 @@ type crypto$key =
928926

929927
declare class crypto$KeyObject {
930928
+asymmetricKeyType?:
931-
| 'rsa'
932-
| 'rsa-pss'
933-
| 'dsa'
934-
| 'ec'
935-
| 'ed25519'
936-
| 'ed448'
937-
| 'x25519'
938-
| 'x448';
929+
'rsa' | 'rsa-pss' | 'dsa' | 'ec' | 'ed25519' | 'ed448' | 'x25519' | 'x448';
939930
+asymmetricKeySize?: number;
940931
+symmetricKeySize?: number;
941932
+type: 'secret' | 'public' | 'private';
@@ -2560,9 +2551,9 @@ declare class http$Agent<+SocketT = net$Socket> {
25602551
sockets: {[name: string]: ReadonlyArray<SocketT>, ...};
25612552
}
25622553

2563-
declare class http$IncomingMessage<SocketT = net$Socket>
2564-
extends stream$Readable
2565-
{
2554+
declare class http$IncomingMessage<
2555+
SocketT = net$Socket,
2556+
> extends stream$Readable {
25662557
headers: Object;
25672558
rawHeaders: Array<string>;
25682559
httpVersion: string;
@@ -2578,9 +2569,9 @@ declare class http$IncomingMessage<SocketT = net$Socket>
25782569
rawTrailers: Array<string>;
25792570
}
25802571

2581-
declare class http$ClientRequest<+SocketT = net$Socket>
2582-
extends stream$Writable
2583-
{
2572+
declare class http$ClientRequest<
2573+
+SocketT = net$Socket,
2574+
> extends stream$Writable {
25842575
abort(): void;
25852576
aborted: boolean;
25862577
+connection: SocketT | null;
@@ -2878,10 +2869,7 @@ declare class net$Socket extends stream$Duplex {
28782869
destroyed: boolean;
28792870
end(
28802871
chunkOrEncodingOrCallback?:
2881-
| Buffer
2882-
| Uint8Array
2883-
| string
2884-
| ((data: any) => void),
2872+
Buffer | Uint8Array | string | ((data: any) => void),
28852873
encodingOrCallback?: string | ((data: any) => void),
28862874
callback?: (data: any) => void,
28872875
): this;
@@ -3211,9 +3199,9 @@ declare module 'perf_hooks' {
32113199
+detail?: T;
32123200
}
32133201

3214-
declare export class PerformanceMeasure<T = unknown>
3215-
extends PerformanceEntry
3216-
{
3202+
declare export class PerformanceMeasure<
3203+
T = unknown,
3204+
> extends PerformanceEntry {
32173205
+entryType: 'measure';
32183206
+detail?: T;
32193207
}
@@ -3398,11 +3386,7 @@ declare module 'querystring' {
33983386
*/
33993387
declare module 'node:sqlite' {
34003388
declare export type SupportedValueType =
3401-
| null
3402-
| number
3403-
| bigint
3404-
| string
3405-
| Uint8Array;
3389+
null | number | bigint | string | Uint8Array;
34063390

34073391
declare export type DatabaseSyncOptions = Readonly<{
34083392
open?: boolean,
@@ -5565,9 +5549,7 @@ declare module 'repl' {
55655549
writer?: (object: any, options?: util$InspectOptions) => string,
55665550
completer?: readline$InterfaceCompleter,
55675551
replMode?:
5568-
| $SymbolReplModeMagic
5569-
| $SymbolReplModeSloppy
5570-
| $SymbolReplModeStrict,
5552+
$SymbolReplModeMagic | $SymbolReplModeSloppy | $SymbolReplModeStrict,
55715553
breakEvalOnSigint?: boolean,
55725554
...
55735555
}): REPLServer;

flow-typed/npm/babel-traverse_v7.x.x.js

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -246,12 +246,7 @@ declare module '@babel/traverse' {
246246
}
247247

248248
declare export type BindingKind =
249-
| 'var'
250-
| 'let'
251-
| 'const'
252-
| 'module'
253-
| 'hoisted'
254-
| 'unknown';
249+
'var' | 'let' | 'const' | 'module' | 'hoisted' | 'unknown';
255250

256251
declare export class Binding {
257252
constructor(opts: {
@@ -435,13 +430,7 @@ declare module '@babel/traverse' {
435430
isBaseType(baseName: string, soft?: boolean): boolean;
436431
couldBeBaseType(
437432
name:
438-
| 'string'
439-
| 'number'
440-
| 'boolean'
441-
| 'any'
442-
| 'mixed'
443-
| 'empty'
444-
| 'void',
433+
'string' | 'number' | 'boolean' | 'any' | 'mixed' | 'empty' | 'void',
445434
): boolean;
446435
baseTypeStrictlyMatches(right: NodePath<>): ?boolean;
447436
isGenericType(genericName: string): boolean;
@@ -1446,8 +1435,7 @@ declare module '@babel/traverse' {
14461435
}>;
14471436

14481437
declare export type VisitNode<-TNode extends BabelNode, TState> =
1449-
| VisitNodeFunction<TNode, TState>
1450-
| VisitNodeObject<TNode, TState>;
1438+
VisitNodeFunction<TNode, TState> | VisitNodeObject<TNode, TState>;
14511439

14521440
declare export type Visitor<TState = void> = Readonly<{
14531441
enter?: VisitNodeFunction<BabelNode, TState>,

flow-typed/npm/babel_v7.x.x.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1293,8 +1293,7 @@ declare module '@babel/template' {
12931293
};
12941294

12951295
declare export type PublicReplacements =
1296-
| {[string]: ?BabelNode}
1297-
| Array<?BabelNode>;
1296+
{[string]: ?BabelNode} | Array<?BabelNode>;
12981297

12991298
declare export type TemplateBuilder<T> = {
13001299
// Build a new builder, merging the given options with the previous ones.

packages/dev-middleware/src/inspector-proxy/CdpDebugLogging.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,7 @@ const debugCDPMessages = require('debug')('Metro:InspectorProxyCDPMessages');
2121
const CDP_MESSAGES_BATCH_DEBUGGING_THROTTLE_MS = 5000;
2222

2323
export type CDPMessageDestination =
24-
| 'DebuggerToProxy'
25-
| 'ProxyToDebugger'
26-
| 'DeviceToProxy'
27-
| 'ProxyToDevice';
24+
'DebuggerToProxy' | 'ProxyToDebugger' | 'DeviceToProxy' | 'ProxyToDevice';
2825

2926
function getCDPLogPrefix(destination: CDPMessageDestination): string {
3027
return util.format(

packages/dev-middleware/src/inspector-proxy/types.js

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -107,16 +107,11 @@ export type GetPagesResponse = {
107107

108108
// Union type for all possible messages sent from device to Inspector Proxy.
109109
export type MessageFromDevice =
110-
| GetPagesResponse
111-
| WrappedEventFromDevice
112-
| DisconnectRequest;
110+
GetPagesResponse | WrappedEventFromDevice | DisconnectRequest;
113111

114112
// Union type for all possible messages sent from Inspector Proxy to device.
115113
export type MessageToDevice =
116-
| GetPagesRequest
117-
| WrappedEventToDevice
118-
| ConnectRequest
119-
| DisconnectRequest;
114+
GetPagesRequest | WrappedEventToDevice | ConnectRequest | DisconnectRequest;
120115

121116
// Page description object that is sent in response to /json HTTP request from debugger.
122117
export type PageDescription = Readonly<{

packages/eslint-plugin-react-native/platform-colors.js

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,14 +58,12 @@ module.exports = {
5858
}
5959
const properties = args[0].properties;
6060
properties.forEach(property => {
61-
if (
62-
!(
63-
property.type === 'Property' &&
64-
(property.value.type === 'Literal' ||
65-
(property.value.type === 'CallExpression' &&
66-
property.value.callee.name === 'PlatformColor'))
67-
)
68-
) {
61+
if (!(
62+
property.type === 'Property' &&
63+
(property.value.type === 'Literal' ||
64+
(property.value.type === 'CallExpression' &&
65+
property.value.callee.name === 'PlatformColor'))
66+
)) {
6967
context.report({
7068
node,
7169
messageId: 'dynamicColorIOSValue',

packages/eslint-plugin-specs/react-native-modules.js

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -91,22 +91,18 @@ function isModuleRequire(node) {
9191
}
9292

9393
const memberExpression = callExpression.callee;
94-
if (
95-
!(
96-
memberExpression.object.type === 'Identifier' &&
97-
memberExpression.object.name === 'TurboModuleRegistry'
98-
)
99-
) {
94+
if (!(
95+
memberExpression.object.type === 'Identifier' &&
96+
memberExpression.object.name === 'TurboModuleRegistry'
97+
)) {
10098
return false;
10199
}
102100

103-
if (
104-
!(
105-
memberExpression.property.type === 'Identifier' &&
106-
(memberExpression.property.name === 'get' ||
107-
memberExpression.property.name === 'getEnforcing')
108-
)
109-
) {
101+
if (!(
102+
memberExpression.property.type === 'Identifier' &&
103+
(memberExpression.property.name === 'get' ||
104+
memberExpression.property.name === 'getEnforcing')
105+
)) {
110106
return false;
111107
}
112108
return true;

packages/jest-preset/jest-preset.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,8 @@ module.exports = {
2828
resolver: require.resolve('./jest/resolver.js'),
2929
transform: {
3030
'^.+\\.(js|ts|tsx)$': 'babel-jest',
31-
'^.+\\.(bmp|gif|jpg|jpeg|mp4|png|psd|svg|webp)$': require.resolve(
32-
'./jest/assetFileTransformer.js',
33-
),
31+
'^.+\\.(bmp|gif|jpg|jpeg|mp4|png|psd|svg|webp)$':
32+
require.resolve('./jest/assetFileTransformer.js'),
3433
},
3534
transformIgnorePatterns: [
3635
'node_modules/(?!((jest-)?react-native|@react-native(-community)?)/)',

packages/metro-config/src/index.flow.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -85,12 +85,10 @@ export function getDefaultConfig(projectRoot: string): ConfigT {
8585
transformer: {
8686
allowOptionalDependencies: true,
8787
assetRegistryPath: 'react-native/asset-registry',
88-
asyncRequireModulePath: require.resolve(
89-
'metro-runtime/src/modules/asyncRequire',
90-
),
91-
babelTransformerPath: require.resolve(
92-
'@react-native/metro-babel-transformer',
93-
),
88+
asyncRequireModulePath:
89+
require.resolve('metro-runtime/src/modules/asyncRequire'),
90+
babelTransformerPath:
91+
require.resolve('@react-native/metro-babel-transformer'),
9492
getTransformOptions: async () => ({
9593
transform: {
9694
experimentalImportSupport: false,

0 commit comments

Comments
 (0)