Skip to content

Commit

Permalink
Enable import/no-anonymous-default-export (#11478)
Browse files Browse the repository at this point in the history
* Enable import/no-anonymous-default-export

* Add names to default exported

* More

* Ignore shims

* Update scripts/build/config.mjs

Co-authored-by: fisker Cheung <lionkay@gmail.com>

* lint  for shims

Co-authored-by: fisker Cheung <lionkay@gmail.com>
  • Loading branch information
sosukesuzuki and fisker committed Sep 10, 2021
1 parent d3d1bc1 commit 2664fcb
Show file tree
Hide file tree
Showing 27 changed files with 31 additions and 26 deletions.
1 change: 1 addition & 0 deletions .eslintrc.js
Expand Up @@ -97,6 +97,7 @@ module.exports = {
},
],
"import/order": "error",
"import/no-anonymous-default-export": "error",

"regexp/match-any": [
"error",
Expand Down
3 changes: 2 additions & 1 deletion scripts/build/config.mjs
Expand Up @@ -162,4 +162,5 @@ const coreBundles = [
...bundle,
}));

export default [...coreBundles, ...parsers];
const configs = [...coreBundles, ...parsers];
export default configs;
2 changes: 1 addition & 1 deletion scripts/build/rollup-plugins/evaluate.mjs
Expand Up @@ -2,7 +2,7 @@ import createEsmUtils from "esm-utils";

const { require } = createEsmUtils(import.meta);

export default function () {
export default function rollupPluginEvaluate() {
return {
name: "evaluate",

Expand Down
2 changes: 1 addition & 1 deletion scripts/build/rollup-plugins/executable.mjs
@@ -1,7 +1,7 @@
import fs from "node:fs";
import path from "node:path";

export default function () {
export default function rollupPluginExecutable() {
let banner;
let entry;
let file;
Expand Down
2 changes: 1 addition & 1 deletion scripts/build/rollup-plugins/replace-module.mjs
@@ -1,4 +1,4 @@
export default function (replacements = {}) {
export default function rollupPluginReplaceModule(replacements = {}) {
return {
name: "replace-module",

Expand Down
3 changes: 2 additions & 1 deletion scripts/build/shims/fs.mjs
@@ -1,3 +1,4 @@
export const existsSync = () => false;
export const readFileSync = () => "";
export default { existsSync, readFileSync };
const fs = { existsSync, readFileSync };
export default fs;
3 changes: 2 additions & 1 deletion scripts/build/shims/os.mjs
@@ -1,5 +1,6 @@
export default {
const os = {
EOL: "\n",
platform: () => "browser",
cpus: () => [{ model: "Prettier" }],
};
export default os;
3 changes: 2 additions & 1 deletion scripts/build/shims/tty.mjs
@@ -1,5 +1,6 @@
export default {
const tty = {
isatty() {
return false;
},
};
export default tty;
2 changes: 1 addition & 1 deletion scripts/release/steps/bump-prettier.js
Expand Up @@ -36,7 +36,7 @@ async function bump({
await writeJson("package.json", pkg, { spaces: 2 });
}

export default async function (params) {
export default async function bumpPrettier(params) {
const { dry, version } = params;

if (dry) {
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/check-git-status.js
@@ -1,6 +1,6 @@
import { runGit } from "../utils.js";

export default async function () {
export default async function checkGitStatus() {
const { stdout: status } = await runGit(["status", "--porcelain"]);

if (status) {
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/generate-bundles.js
@@ -1,7 +1,7 @@
import chalk from "chalk";
import { runYarn, logPromise, readJson } from "../utils.js";

export default async function ({ version }) {
export default async function generateBundles({ version }) {
await logPromise("Generating bundles", runYarn(["build", "--purge-cache"]));

const builtPkg = await readJson("dist/package.json");
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/install-dependencies.js
Expand Up @@ -13,6 +13,6 @@ async function install() {
}
}

export default function () {
export default function installDependencies() {
return logPromise("Installing NPM dependencies", install());
}
2 changes: 1 addition & 1 deletion scripts/release/steps/post-publish-steps.js
Expand Up @@ -43,7 +43,7 @@ function twitterAnnouncement() {
`);
}

export default async function () {
export default async function postPublishSteps() {
const steps = [await checkSchema(), twitterAnnouncement()].filter(Boolean);

console.log(chalk.bold.green("The script has finished!\n"));
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/publish-to-npm.js
Expand Up @@ -27,7 +27,7 @@ async function retryNpmPublish() {
}
}

export default async function ({ dry, version }) {
export default async function publishToNpm({ dry, version }) {
if (dry) {
return;
}
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/push-to-git.js
Expand Up @@ -7,7 +7,7 @@ async function pushGit({ version }) {
await runGit(["push", "--tags"]);
}

export default function (params) {
export default function pushToGit(params) {
if (params.dry) {
return;
}
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/run-tests.js
@@ -1,6 +1,6 @@
import { runYarn, logPromise } from "../utils.js";

export default async function () {
export default async function runTests() {
await logPromise("Running linter", runYarn("lint:eslint"));
await logPromise("Running Prettier on docs", runYarn("lint:prettier"));
await logPromise("Running tests", runYarn("test"));
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/update-changelog.js
Expand Up @@ -42,7 +42,7 @@ async function getChangelogForPatch({ version, previousVersion }) {
return changelog;
}

export default async function ({ version, previousVersion }) {
export default async function updateChangelog({ version, previousVersion }) {
const semverDiff = semver.diff(version, previousVersion);

if (semverDiff !== "patch") {
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/update-dependents-count.js
Expand Up @@ -71,7 +71,7 @@ function formatNumber(value) {
return Math.floor(value / 1e5) / 10 + " million";
}

export default async function () {
export default async function updateDependentsCount() {
try {
await update();
} catch (error) {
Expand Down
2 changes: 1 addition & 1 deletion scripts/release/steps/update-version.js
Expand Up @@ -32,6 +32,6 @@ async function bump({ version }) {
});
}

export default async function (params) {
export default async function updateVersion(params) {
await logPromise("Bumping version", bump(params));
}
2 changes: 1 addition & 1 deletion scripts/release/steps/validate-new-version.js
@@ -1,7 +1,7 @@
import chalk from "chalk";
import semver from "semver";

export default function ({ version, previousVersion }) {
export default function validateNewVersion({ version, previousVersion }) {
if (!semver.valid(version)) {
throw new Error("Invalid version specified");
}
Expand Down
2 changes: 1 addition & 1 deletion website/playground/BottomBar.js
Expand Up @@ -3,7 +3,7 @@ import * as ReactDOM from "react-dom";

const root = document.getElementById("bottom-bar");

export default function ({ left, right }) {
export default function BottomBar({ left, right }) {
return ReactDOM.createPortal(
<React.Fragment>
<div className="bottom-bar-buttons">{left}</div>
Expand Down
2 changes: 1 addition & 1 deletion website/playground/EditorState.js
Expand Up @@ -3,7 +3,7 @@ import * as React from "react";
import { stateToggler, shallowEqual } from "./helpers.js";
import * as storage from "./storage.js";

export default class extends React.Component {
export default class EditorState extends React.Component {
constructor() {
super();
this.state = {
Expand Down
2 changes: 1 addition & 1 deletion website/playground/VersionLink.js
Expand Up @@ -3,7 +3,7 @@ import * as ReactDOM from "react-dom";

const root = document.getElementById("version");

export default function ({ version }) {
export default function VersionLink({ version }) {
const match = version.match(/^pr-(\d+)$/);
let href;
if (match) {
Expand Down
2 changes: 1 addition & 1 deletion website/playground/WorkerApi.js
@@ -1,4 +1,4 @@
export default function (source) {
export default function WorkerApi(source) {
const worker = new Worker(source);
let counter = 0;
const handlers = {};
Expand Down
2 changes: 1 addition & 1 deletion website/playground/codeSamples.js
@@ -1,4 +1,4 @@
export default function (parser) {
export default function getCodeSamples(parser) {
switch (parser) {
case "babel":
case "espree":
Expand Down
2 changes: 1 addition & 1 deletion website/playground/sidebar/SidebarOptions.js
Expand Up @@ -4,7 +4,7 @@ import groupBy from "lodash/groupBy";
import { SidebarCategory } from "./components.js";
import Option from "./options.js";

export default function ({
export default function SidebarOptions({
categories,
availableOptions,
optionValues,
Expand Down
2 changes: 1 addition & 1 deletion website/playground/sidebar/options.js
Expand Up @@ -42,7 +42,7 @@ export function NumberOption({ option, value, onChange }) {
);
}

export default function (props) {
export default function Option(props) {
switch (props.option.type) {
case "boolean":
return <BooleanOption {...props} />;
Expand Down

0 comments on commit 2664fcb

Please sign in to comment.