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.

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

[pull] master from airbnb:master #41

Open
wants to merge 59 commits into
base: master
Choose a base branch
from
Open

[pull] master from airbnb:master #41

wants to merge 59 commits into from

Conversation

pull[bot]
Copy link

@pull pull bot commented Aug 10, 2022

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

This rule was deprecated since ESLint v3.3.0 and replaced by the
`func-call-spacing` rule.
https://eslint.org/docs/latest/rules/no-spaced-func

`func-call-spacing` was enabled in this package long ago:
27dcb99

So there's no need for it to stay in the ruleset.

Besides, it is causing some tricky issues when overriding the rules for
TypeScript support (iamturns/eslint-config-airbnb-typescript#246)
@pull pull bot added the ⤵️ pull label Aug 10, 2022
ljharb and others added 25 commits September 18, 2022 22:08
2.1 Use const for all of your references; avoid using var. eslint: prefer-const, no-const-assign
The change itself has already been made in f3d3a07
(tracked in/acording to #2421 (comment)).


#2421
Remove Math.pow from `no-restricted-properties` and use `prefer-exponentiation-operator` instead.
ljharb and others added 30 commits May 15, 2023 10:21
Co-authored-by: Jordan Harband <ljharb@gmail.com>
The recommendation is to declare the variables, classes and functions before. So the contrasting negative example should say after.
This removes the `object.entries` package and uses the widely available built-in native `Object.entries`.
Signed-off-by: Kim Minh Thắng <thangved02@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.