Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Explain ASI issues aren't always auto-fixed #14174

Merged
merged 2 commits into from
Feb 18, 2023
Merged

Explain ASI issues aren't always auto-fixed #14174

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
602ea00
Explain ASI issues aren't always auto-fixed
theScottyJam Jan 14, 2023
581e46b
Apply change to playground
fisker Feb 18, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
18 changes: 18 additions & 0 deletions docs/rationale.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,24 @@ With a semicolon in front of that `[` such issues never happen. It makes the lin

This practice is also common in [standard] which uses a semicolon-free style.

Note that if your program currently has a semicolon-related bug in it, Prettier _will not_ auto-fix the bug for you. Remember, Prettier only reformats code, it does not change the behavior of the code. Take this buggy piece of code as an example, where the developer forgot to place a semicolon before the `(`:

<!-- prettier-ignore -->
```js
console.log('Running a background task')
(async () => {
await doBackgroundWork()
})()
```

If you feed this into Prettier, it will not alter the behavior of this code, instead, it will reformat it in a way that shows how this code will actually behave when ran.

```js
console.log("Running a background task")(async () => {
await doBackgroundWork();
})();
```

[standard]: https://standardjs.com/rules.html#semicolons

### Print width
Expand Down