quick-lint-js

quick-lint-js copied to clipboard

12$: website: svg icons render incorrectly on Chrome/Edge/Safari

5

strager

[doc] [request] Add popular CI installation instructions

1

When reading 📖 about https://golangci-lint.run/usage/install/ a few weeks ago, I thought 💭 it was interesting 🤨 how they promote running it in CI and giving instructions for that. Do you...

vegerot

https://cdn.discordapp.com/attachments/835712135962296330/1006447947497623602/unknown.png

strager

e.g. `before` https://mochajs.org/#hooks

strager

```javascript import React from "react"; // the following should warn about missing 'export' function MyComponent() { return hello; } ```

strager

The following code is missing the export list, but quick-lint-js thinks it's missing the `from` keyword: ```js import from "react"; ``` Improve the error message to suggest writing an export...

strager

strager

Optimize variable lookup

8

About 30% of lex+parse+lint time for `three.min.js` is spent comparing identifier names in `linter` (since renamed to `variable_analyzer`): Let's optimize this for some nice perf win.

strager

8$: Extra character `,` on end of useState() gives mediocre error message.

5

Doesn't tell me to remove extra character `,` on the end.  This should give an error saying unexpected token and missing semicolon like below: 

kyle-wannacott

Error if calling a string/template/number literal

20

```js let randomVar = 5(); ``` 5 is called as if it was a function. We should report an error for trivial cases like this. In the future, we might...

strager