Skip to content

Commit

Permalink
Fixup formatting
Browse files Browse the repository at this point in the history
  • Loading branch information
@lukastaegert
lukastaegert committed Apr 3, 2020
1 parent 1f9a113 commit 751113b
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 27 deletions.
48 changes: 23 additions & 25 deletions src/Module.ts
View file
Expand Up @@ -5,7 +5,7 @@ import extractAssignedNames from 'rollup-pluginutils/src/extractAssignedNames';
import {
createHasEffectsContext,
createInclusionContext,
InclusionContext,
InclusionContext
} from './ast/ExecutionContext';
import ExportAllDeclaration from './ast/nodes/ExportAllDeclaration';
import ExportDefaultDeclaration from './ast/nodes/ExportDefaultDeclaration';
Expand Down Expand Up @@ -42,7 +42,7 @@ import {
ResolvedIdMap,
RollupError,
RollupWarning,
TransformModuleJSON,
TransformModuleJSON
} from './rollup/types';
import { error, Errors } from './utils/error';
import getCodeFrame from './utils/getCodeFrame';
Expand Down Expand Up @@ -123,7 +123,7 @@ export interface AstContext {
export const defaultAcornOptions: acorn.Options = {
ecmaVersion: 2020,
preserveParens: false,
sourceType: 'module',
sourceType: 'module'
};

function tryParse(module: Module, Parser: typeof acorn.Parser, acornOptions: acorn.Options) {
Expand All @@ -132,7 +132,7 @@ function tryParse(module: Module, Parser: typeof acorn.Parser, acornOptions: aco
...defaultAcornOptions,
...acornOptions,
onComment: (block: boolean, text: string, start: number, end: number) =>
module.comments.push({ block, text, start, end }),
module.comments.push({ block, text, start, end })
});
} catch (err) {
let message = err.message.replace(/ \(\d+:\d+\)$/, '');
Expand All @@ -145,7 +145,7 @@ function tryParse(module: Module, Parser: typeof acorn.Parser, acornOptions: aco
{
code: 'PARSE_ERROR',
message,
parserError: err,
parserError: err
},
err.pos
);
Expand All @@ -164,15 +164,15 @@ function handleMissingExport(
message: `'${exportName}' is not exported by ${relativeId(
importedModule
)}, imported by ${relativeId(importingModule.id)}`,
url: `https://rollupjs.org/guide/en/#error-name-is-not-exported-by-module`,
url: `https://rollupjs.org/guide/en/#error-name-is-not-exported-by-module`
},
importerStart!
);
}

const MISSING_EXPORT_SHIM_DESCRIPTION: ExportDescription = {
identifier: null,
localName: MISSING_EXPORT_SHIM_VARIABLE,
localName: MISSING_EXPORT_SHIM_VARIABLE
};

function getVariableForExportNameRecursive(
Expand Down Expand Up @@ -290,17 +290,17 @@ export default class Module {
loc: {
column: location.column,
file: this.id,
line: location.line,
line: location.line
},
message: `Error when using sourcemap for reporting an error: ${e.message}`,
pos,
pos
});
}

props.loc = {
column: location.column,
file: this.id,
line: location.line,
line: location.line
};
props.frame = getCodeFrame(this.originalCode, location.line, location.column);
}
Expand Down Expand Up @@ -342,7 +342,7 @@ export default class Module {
return error({
code: Errors.SYNTHETIC_NAMED_EXPORTS_NEED_DEFAULT,
id: this.id,
message: `Modules with 'syntheticNamedExports' need a default export.`,
message: `Modules with 'syntheticNamedExports' need a default export.`
});
}
return this.defaultExport;
Expand Down Expand Up @@ -632,7 +632,7 @@ export default class Module {
sourcemapChain,
syntheticNamedExports,
transformDependencies,
transformFiles,
transformFiles
}: TransformModuleJSON & {
alwaysRemovedCode?: [number, number][];
transformFiles?: EmittedFile[] | undefined;
Expand Down Expand Up @@ -678,7 +678,7 @@ export default class Module {

this.magicString = new MagicString(code, {
filename: (this.excludeFromSourcemap ? null : fileName)!, // don't include plugin helpers in sourcemap
indentExclusionRanges: [],
indentExclusionRanges: []
});
for (const [start, end] of this.alwaysRemovedCode) {
this.magicString.remove(start, end);
Expand All @@ -704,7 +704,7 @@ export default class Module {
includeDynamicImport: this.includeDynamicImport.bind(this),
includeExternalReexportNamespace: this.includeExternalReexportNamespace.bind(this),
includeVariable: this.includeVariable.bind(this),
isCrossChunkImport: (importDescription) =>
isCrossChunkImport: importDescription =>
(importDescription.module as Module).chunk !== this.chunk,
magicString: this.magicString,
module: this,
Expand All @@ -722,7 +722,7 @@ export default class Module {
this.graph.treeshakingOptions.unknownGlobalSideEffects)!,
usesTopLevelAwait: false,
warn: this.warn.bind(this),
warnDeprecation: this.graph.warnDeprecation.bind(this.graph),
warnDeprecation: this.graph.warnDeprecation.bind(this.graph)
};

this.scope = new ModuleScope(this.graph.scope, this.astContext);
Expand All @@ -741,7 +741,7 @@ export default class Module {
ast: this.esTreeAst,
code: this.code,
customTransformCache: this.customTransformCache,
dependencies: Array.from(this.dependencies).map((module) => module.id),
dependencies: Array.from(this.dependencies).map(module => module.id),
id: this.id,
moduleSideEffects: this.moduleSideEffects,
originalCode: this.originalCode,
Expand All @@ -750,7 +750,7 @@ export default class Module {
sourcemapChain: this.sourcemapChain,
syntheticNamedExports: this.syntheticNamedExports,
transformDependencies: this.transformDependencies,
transformFiles: this.transformFiles,
transformFiles: this.transformFiles
};
}

Expand Down Expand Up @@ -811,7 +811,7 @@ export default class Module {

this.exports.default = {
identifier: node.variable.getAssignedVariableName(),
localName: 'default',
localName: 'default'
};
} else if (node instanceof ExportAllDeclaration) {
// export * from './other'
Expand All @@ -831,7 +831,7 @@ export default class Module {
specifier.type === NodeType.ExportNamespaceSpecifier ? '*' : specifier.local.name,
module: null as any, // filled in later,
source,
start: specifier.start,
start: specifier.start
};
}
} else if (node.declaration) {
Expand Down Expand Up @@ -872,7 +872,7 @@ export default class Module {
return this.error(
{
code: 'DUPLICATE_IMPORT',
message: `Duplicated import '${localName}'`,
message: `Duplicated import '${localName}'`
},
specifier.start
);
Expand All @@ -890,7 +890,7 @@ export default class Module {
module: null as any, // filled in later
name,
source,
start: specifier.start,
start: specifier.start
};
}
}
Expand All @@ -910,9 +910,7 @@ export default class Module {
}

private includeDynamicImport(node: ImportExpression) {
const resolution = (this.dynamicImports.find(
(dynamicImport) => dynamicImport.node === node
) as {
const resolution = (this.dynamicImports.find(dynamicImport => dynamicImport.node === node) as {
resolution: string | Module | ExternalModule | undefined;
}).resolution;
if (resolution instanceof Module) {
Expand Down Expand Up @@ -952,7 +950,7 @@ export default class Module {
code: 'SHIMMED_EXPORT',
exporter: relativeId(this.id),
exportName: name,
message: `Missing export "${name}" has been shimmed in module ${relativeId(this.id)}.`,
message: `Missing export "${name}" has been shimmed in module ${relativeId(this.id)}.`
});
this.exports[name] = MISSING_EXPORT_SHIM_DESCRIPTION;
}
Expand Down
4 changes: 2 additions & 2 deletions src/ast/variables/NamespaceVariable.ts
View file
Expand Up @@ -78,7 +78,7 @@ export default class NamespaceVariable extends Variable {
const n = options.compact ? '' : '\n';
const t = options.indent;

const members = Object.keys(this.memberVariables).map((name) => {
const members = Object.keys(this.memberVariables).map(name => {
const original = this.memberVariables[name];

if (this.referencedEarly || original.isReassigned) {
Expand All @@ -102,7 +102,7 @@ export default class NamespaceVariable extends Variable {
let output = `{${n}${members.join(`,${n}`)}${n}}`;
if (this.reexportedNamespaceVariables.length) {
output = `/*#__PURE__*/Object.assign(${this.reexportedNamespaceVariables
.map((v) => v.getName())
.map(v => v.getName())
.join(', ')}, ${output})`;
}
if (this.syntheticNamedExports) {
Expand Down

0 comments on commit 751113b

Please sign in to comment.