Skip to content
This repository was archived by the owner on Jan 26, 2019. It is now read-only.

Removed references to eslint and updated README #9

Merged
merged 1 commit into from
Jan 2, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,4 @@ template/src/__tests__/__snapshots__/
lerna-debug.log
npm-debug.log
/.changelog
yarn.lock
1 change: 0 additions & 1 deletion packages/react-scripts/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
},
"files": [
".babelrc",
".eslintrc",
"config",
"bin",
"scripts",
Expand Down
10 changes: 6 additions & 4 deletions packages/react-scripts/scripts/start.js
Original file line number Diff line number Diff line change
Expand Up @@ -127,10 +127,12 @@ function setupCompiler(host, port, protocol) {
console.log(message);
console.log();
});
// Teach some ESLint tricks.
console.log('You may use special comments to disable some warnings.');
console.log('Use ' + chalk.yellow('// eslint-disable-next-line') + ' to ignore the next line.');
console.log('Use ' + chalk.yellow('/* eslint-disable */') + ' to ignore all warnings in a file.');
// Teach some TSLint tricks.
console.log('You may use special comments to disable some warnings.');
console.log('Use ' + chalk.yellow('tslint:disable-line') + ' to disable this line.');
console.log('Use ' + chalk.yellow('tslint:disable-next-line') + ' to ignore the rules on next line.');
console.log('Use ' + chalk.yellow('tslint:disable ') + ' to disable linting for rest of file.');
console.log('Use ' + chalk.yellow('tslint:enable ') + ' to enable linting for rest of file.');
}
});
}
Expand Down
33 changes: 4 additions & 29 deletions packages/react-scripts/template/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ See the section about [deployment](#deployment) for more information.

If you aren’t satisfied with the build tool and configuration choices, you can `eject` at any time. This command will remove the single build dependency from your project.

Instead, it will copy all the configuration files and the transitive dependencies (Webpack, Babel, ESLint, etc) right into your project so you have full control over them. All of the commands except `eject` will still work, but they will point to the copied scripts so you can tweak them. At this point you’re on your own.
Instead, it will copy all the configuration files and the transitive dependencies (Webpack, Babel, TSLint, etc) right into your project so you have full control over them. All of the commands except `eject` will still work, but they will point to the copied scripts so you can tweak them. At this point you’re on your own.

You don’t have to ever use `eject`. The curated feature set is suitable for small and middle deployments, and you shouldn’t feel obligated to use this feature. However we understand that this tool wouldn’t be useful if you couldn’t customize it when you are ready for it.

Expand All @@ -170,36 +170,11 @@ To configure the syntax highlighting in your favorite text editor, head to the [

>Note: this feature is available with `[email protected]` and higher.

Some editors, including Sublime Text, Atom, and Visual Studio Code, provide plugins for ESLint.
Some editors, including Sublime Text, Atom, and Visual Studio Code, provide plugins for TSLint.

They are not required for linting. You should see the linter output right in your terminal as well as the browser console. However, if you prefer the lint results to appear right in your editor, there are some extra steps you can do.

You would need to install an ESLint plugin for your editor first.

>**A note for Atom `linter-eslint` users**

>If you are using the Atom `linter-eslint` plugin, make sure that **Use global ESLint installation** option is checked:

><img src="http://i.imgur.com/yVNNHJM.png" width="300">

Then add this block to the `package.json` file of your project:

```js
{
// ...
"eslintConfig": {
"extends": "react-app"
}
}
```

Finally, you will need to install some packages *globally*:

```sh
npm install -g [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]
```

We recognize that this is suboptimal, but it is currently required due to the way we hide the ESLint dependency. The ESLint team is already [working on a solution to this](https://github.com/eslint/eslint/issues/3458) so this may become unnecessary in a couple of months.
You would need to install an TSLint plugin for your editor first.

## Changing the Page `<title>`

Expand Down Expand Up @@ -448,7 +423,7 @@ const $ = window.$;

This makes it obvious you are using a global variable intentionally rather than because of a typo.

Alternatively, you can force the linter to ignore any line by adding `// eslint-disable-line` after it.
Alternatively, you can force the linter to ignore any line by adding `// tslint:disable-line` after it.

## Adding Bootstrap

Expand Down