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

Update eslint (major) #1851

Merged
merged 9 commits into from
Oct 11, 2019
Merged

Update eslint (major) #1851

merged 9 commits into from
Oct 11, 2019

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jun 15, 2019

This PR contains the following updates:

Package Type Update Change
@typescript-eslint/eslint-plugin devDependencies major 1.13.0 -> 2.3.3
@typescript-eslint/parser devDependencies major 1.13.0 -> 2.3.3
eslint (source) devDependencies major 5.16.0 -> 6.5.1
eslint-config-airbnb devDependencies major 17.1.1 -> 18.0.1
eslint-config-airbnb-base devDependencies major 13.2.0 -> 14.0.0
eslint-config-prettier devDependencies major 4.3.0 -> 6.4.0

Release Notes

typescript-eslint/typescript-eslint

v2.3.3

Compare Source

Bug Fixes
  • eslint-plugin: [class-name-casing] allow unicode letters (#​1043) (47895c0)
  • eslint-plugin: [efrt] support constructor arguments (#​1021) (60943e6)
  • experimental-utils: remove Rule.meta.extraDescription (#​1036) (192e23d)

v2.3.2

Compare Source

Bug Fixes
  • eslint-plugin: [class-name-casing] allow unicode letters (#​1043) (47895c0)
  • eslint-plugin: [efrt] support constructor arguments (#​1021) (60943e6)
  • experimental-utils: remove Rule.meta.extraDescription (#​1036) (192e23d)

v2.3.1

Compare Source

Bug Fixes
  • eslint-plugin: [no-unnec-type-arg] undefined symbol crash (#​1007) (cdf9294)
  • typescript-estree: correct ClassDeclarationBase type (#​1008) (8ce3a81)
  • typescript-estree: handle optional computed prop w/o type (#​1026) (95c13fe)

v2.3.0

Compare Source

Bug Fixes
  • typescript-estree: ImportDeclaration.specifier to Literal (#​974) (2bf8231)
Features
  • eslint-plugin: [explicit-member-accessibility] add support of "ignoredMethodNames" (#​895) (46ee4c9)
  • eslint-plugin: [no-floating-promises] Add ignoreVoid option (#​796) (6a55921)
  • eslint-plugin: [no-magic-numbers] add ignoreReadonlyClassProperties option (#​938) (aeea4cd)
  • eslint-plugin: [strict-boolean-expressions] Add allowNullable option (#​794) (c713ca4)
  • eslint-plugin: add no-unnecessary-condition rule (#​699) (5715482)

v2.2.0

Compare Source

Bug Fixes
  • typescript-estree: ImportDeclaration.specifier to Literal (#​974) (2bf8231)
Features
  • eslint-plugin: [explicit-member-accessibility] add support of "ignoredMethodNames" (#​895) (46ee4c9)
  • eslint-plugin: [no-floating-promises] Add ignoreVoid option (#​796) (6a55921)
  • eslint-plugin: [no-magic-numbers] add ignoreReadonlyClassProperties option (#​938) (aeea4cd)
  • eslint-plugin: [strict-boolean-expressions] Add allowNullable option (#​794) (c713ca4)
  • eslint-plugin: add no-unnecessary-condition rule (#​699) (5715482)

v2.1.0

Compare Source

Bug Fixes
  • eslint-plugin: [efrt] allowExpressions - check functions in class field properties (#​952) (f1059d8)
  • eslint-plugin: [expl-member-a11y] fix parameter properties (#​912) (ccb98d8)
  • eslint-plugin: [prefer-readonly] add handling for destructuring assignments (e011e90)
Features
  • eslint-plugin: add brace-style extension (e01dc5f)

v2.0.0

Compare Source

Bug Fixes
  • eslint-plugin: [member-naming] should match constructor args (#​771) (b006667)
  • eslint-plugin: [no-inferrable-types] ignore optional props (#​918) (a4e625f)
  • eslint-plugin: [promise-function-async] Allow async get/set (#​820) (cddfdca)
  • eslint-plugin: [require-await] Allow concise arrow function bodies (#​826) (29fddfd)
  • eslint-plugin: [typedef] don't flag destructuring when variables is disabled (#​819) (5603473)
  • eslint-plugin: [typedef] handle AssignmentPattern inside TSParameterProperty (#​923) (6bd7f2d)
  • eslint-plugin: [unbound-method] Allow typeof expressions (Fixes #​692) (#​904) (344bafe)
  • eslint-plugin: [unbound-method] false positive in equality comparisons (#​914) (29a01b8)
  • eslint-plugin: [unified-signatures] type comparison and exported nodes (#​839) (580eceb)
  • eslint-plugin: readme typo (#​867) (5eb40dc)
  • typescript-estree: improve missing project file error msg (#​866) (8f3b0a8), closes #​853
Features
  • [no-unnecessary-type-assertion] allow as const arrow functions (#​876) (14c6f80)
  • eslint-plugin: [expl-func-ret-type] make error loc smaller (#​919) (65eb993)
  • eslint-plugin: [no-type-alias] support tuples (#​775) (c68e033)
  • eslint-plugin: add quotes extension (9f82099)
  • typescript-estree: Accept a glob pattern for options.project (#​806) (9e5f21e)
eslint/eslint

v6.5.1

Compare Source

  • 0d3d7d9 Docs: fix typo in no-magic-numbers (#​12345) (Josiah Rooney)
  • 447ac87 Fix: no-useless-rename handles ExperimentalRestProperty (fixes #​12335) (#​12339) (Kai Cataldo)
  • b6ff73c Sponsors: Sync README with website (ESLint Jenkins)

v6.5.0

Compare Source

  • 73596cb Update: Add enforceForSwitchCase option to use-isnan (#​12106) (Milos Djermanovic)
  • d592a24 Fix: exclude \u000d so new line won't convert to text (fixes #​12027) (#​12031) (zamboney)
  • e85d27a Fix: no-regex-spaces false positives and invalid autofix (fixes #​12226) (#​12231) (Milos Djermanovic)
  • b349bf7 Fix: prefer-named-capture-group incorrect locations (fixes #​12233) (#​12247) (Milos Djermanovic)
  • 7dc1ea9 Fix: no-useless-return autofix removes comments (#​12292) (Milos Djermanovic)
  • 0e68677 Fix: no-extra-bind autofix removes comments (#​12293) (Milos Djermanovic)
  • 6ad7e86 Fix: no-extra-label autofix removes comments (#​12298) (Milos Djermanovic)
  • acec201 Fix: no-undef-init autofix removes comments (#​12299) (Milos Djermanovic)
  • d89390b Fix: use async reading of stdin in bin/eslint.js (fixes #​12212) (#​12230) (Barrie Treloar)
  • 334ca7c Update: no-useless-rename also reports default values (fixes #​12301) (#​12322) (Kai Cataldo)
  • 41bfe91 Update: Fix handling of chained new expressions in new-parens (#​12303) (Milos Djermanovic)
  • 160b7c4 Chore: add autofix npm script (#​12330) (Kai Cataldo)
  • 04b6adb Chore: enable eslint-plugin-jsdoc (refs #​11146) (#​12332) (Kai Cataldo)
  • 9b86167 Docs: Add new ES environments to Configuring ESLint (#​12289) (Milos Djermanovic)
  • c9aeab2 Docs: Add supported ECMAScript version to README (#​12290) (Milos Djermanovic)
  • 8316e7b Fix: no-useless-rename autofix removes comments (#​12300) (Milos Djermanovic)
  • 29c12f1 Chore: cache results in runtime-info (#​12320) (Kai Cataldo)
  • f5537b2 Fix: prefer-numeric-literals autofix removes comments (#​12313) (Milos Djermanovic)
  • 11ae6fc Update: Fix call, new and member expressions in no-extra-parens (#​12302) (Milos Djermanovic)
  • a7894eb New: add --env-info flag to CLI (#​12270) (Kai Cataldo)
  • 61392ff Sponsors: Sync README with website (ESLint Jenkins)
  • 2c6bf8e Docs: English fix (#​12306) (Daniel Nixon)
  • 6f11877 Sponsors: Sync README with website (ESLint Jenkins)
  • 2e202ca Docs: fix links in array-callback-return (#​12288) (Milos Djermanovic)
  • e39c631 Docs: add example for CLIEngine#executeOnText 3rd arg (#​12286) (Kai Cataldo)
  • d4f9a16 Update: add support for JSXFragments in indent rule (fixes #​12208) (#​12210) (Kai Cataldo)
  • c6af95f Sponsors: Sync README with website (ESLint Jenkins)
  • 8cadd52 Sponsors: Sync README with website (ESLint Jenkins)
  • f9fc695 Chore: enable default-param-last (#​12244) (薛定谔的猫)
  • 9984c3e Docs: Update README team and sponsors (ESLint Jenkins)

v6.4.0

Compare Source

  • e915fff Docs: Improve examples and clarify default option (#​12067) (Yuping Zuo)
  • 540296f Update: enforceForClassMembers option to accessor-pairs (fixes #​12063) (#​12192) (Milos Djermanovic)
  • d3c2334 Update: flag nested block with declaration as error (#​12193) (David Waller)
  • b2498d2 Update: Fix handling of property names in no-self-assign (#​12105) (Milos Djermanovic)
  • 1ee61b0 Update: enforceForClassMembers computed-property-spacing (fixes #​12049) (#​12214) (Milos Djermanovic)
  • 520c922 Docs: Added naming convention details to plugin usage (#​12202) (Henrique Barcelos)
  • f826eab Fix: Allow line comment exception in object-curly-spacing (fixes #​11902) (#​12216) (Milos Djermanovic)
  • db2a29b Update: indentation of comment followed by semicolon (fixes #​12232) (#​12243) (Kai Cataldo)
  • ae17d1c Fix: no-sequences is reporting incorrect locations (#​12241) (Milos Djermanovic)
  • 365331a Fix: object-shorthand providing invalid fixes for typescript (#​12260) (Brad Zacher)
  • 1c921c6 New: add no-import-assign (fixes #​12237) (#​12252) (Toru Nagashima)
  • 3be04fd New: Add prefer-regex-literals rule (fixes #​12238) (#​12254) (Milos Djermanovic)
  • 37c0fde Update: Report global Atomics calls in no-obj-calls (fixes #​12234) (#​12258) (Milos Djermanovic)
  • 985c9e5 Fix: space-before-function-paren autofix removes comments (fixes #​12259) (#​12264) (Milos Djermanovic)
  • 01da7d0 Fix: eqeqeq rule reports incorrect locations (#​12265) (Milos Djermanovic)
  • 319e4d8 Docs: adding finally example (#​12256) (Jens Melgaard)
  • d52328f Docs: fix no-sequences with examples (#​12239) (Milos Djermanovic)
  • a41fdc0 Fix: Remove autofixer for no-unsafe-negation (#​12157) (Milos Djermanovic)
  • e38f5fd Update: fix no-octal-escape false negatives after \0 (#​12079) (Milos Djermanovic)
  • 9418fbe Sponsors: Sync README with website (ESLint Jenkins)
  • acc5ec5 Sponsors: Sync README with website (ESLint Jenkins)
  • 460c5ad Sponsors: Sync README with website (ESLint Jenkins)
  • 0313441 New: add rule default-param-last (fixes #​11361) (#​12188) (Chiawen Chen)
  • 7621f5d Update: add more specific linting messages to space-in-parens (#​11121) (Che Fisher)
  • 21eb904 Fix: basePath of OverrideTester (fixes #​12032) (#​12205) (Toru Nagashima)
  • 86e5e65 Sponsors: Sync README with website (ESLint Jenkins)
  • 2b1a13f Fix: no-extra-boolean-cast reports wrong negation node (fixes #​11324) (#​12197) (Milos Djermanovic)
  • ba8c2aa Sponsors: Sync README with website (ESLint Jenkins)
  • a0a9746 Docs: Fix link in no-irregular-whitespace.md (#​12196) (Timo Tijhof)
  • e10eeba Fix: quotes autofix produces syntax error with octal escape sequences (#​12118) (Milos Djermanovic)

v6.3.0

Compare Source

v6.2.2

Compare Source

  • 0e0b784 Upgrade: espree@^6.1.1 (#​12158) (Kevin Partington)
  • 04e859f Sponsors: Sync README with website (ESLint Jenkins)
  • 34783d1 Sponsors: Sync README with website (ESLint Jenkins)
  • b809e72 Docs: Update README team and sponsors (ESLint Jenkins)

v6.2.1

Compare Source

  • 8c021b5 Upgrade: eslint-utils 1.4.2 (#​12131) (Toru Nagashima)
  • e82388b Sponsors: Sync README with website (ESLint Jenkins)
  • 4aeeeed Docs: update docs for ecmaVersion 2020 (#​12120) (silverwind)
  • 6886148 Docs: Add duplicate keys limitation to accessor-pairs (#​12124) (Milos Djermanovic)

v6.2.0

Compare Source

  • fee6acb Update: support bigint and dynamic import (refs #​11803) (#​11983) (Toru Nagashima)
  • afd8012 New: noInlineConfig setting (refs eslint/rfcs#​22) (#​12091) (Toru Nagashima)
  • 3d12378 Update: Fix accessor-pairs to enforce pairs per property in literals (#​12062) (Milos Djermanovic)
  • 8cd00b3 New: function-call-argument-newline (#​12024) (finico)
  • 30ebf92 Fix: prefer-template autofix produces syntax error with octal escapes (#​12085) (Milos Djermanovic)
  • 13c3988 Fix: Check literal type explicitly in dot-notation (#​12095) (Milos Djermanovic)
  • 3e5ceca Fix: Handle empty string property names in getFunctionNameWithKind (#​12104) (Milos Djermanovic)
  • 9a043ff Fix: no-duplicate-case false positives on Object.prototype keys (#​12107) (Milos Djermanovic)
  • fe631af Chore: minor typo fix (#​12112) (James George)
  • 4cb7877 Fix: fix no-extra-parens ignores some nodes (#​11909) (Pig Fang)
  • 2dc23b8 Update: fix no-dupe-keys false negatives on empty string names (#​12069) (Milos Djermanovic)
  • 19ab666 Fix: yoda exceptRange false positives on empty string property names (#​12071) (Milos Djermanovic)
  • d642150 Update: Check empty string property names in sort-keys (#​12073) (Milos Djermanovic)
  • acce6de Fix: class-methods-use-this reports 'undefined' names (#​12103) (Milos Djermanovic)
  • 92ec2cb Fix: Allow bind call with a single spread element in no-extra-bind (#​12088) (Milos Djermanovic)
  • bfdb0c9 Fix: no-extra-boolean-cast invalid autofix for Boolean() without args (#​12076) (Milos Djermanovic)
  • 34ccc0c Chore: Remove TDZ scope type condition from no-unused-vars (#​12055) (Milos Djermanovic)
  • 01d38ce Docs: Remove TDZ scope from the scope manager interface documentation (#​12054) (Milos Djermanovic)
  • 1aff8fc Update: warn about mixing ternary and logical operators (fixes #​11704) (#​12001) (Karthik Priyadarshan)
  • 11be2f8 Docs: do not recommend global-installed usage (#​12016) (薛定谔的猫)
  • cf31dab Fix: no-restricted-syntax - correct the schema (#​12051) (Brad Zacher)
  • fbec99e Update: fix class-methods-use-this false negatives with exceptMethods (#​12077) (Milos Djermanovic)
  • fb08b7c Docs: Remove readonly/writable global logic from no-undef (fixes #​11963) (#​12053) (Milos Djermanovic)
  • 5b5934b Sponsors: Sync README with website (ESLint Jenkins)
  • 9156760 Sponsors: Sync README with website (ESLint Jenkins)
  • f5e0cc4 Update: Check computed method keys in no-extra-parens (#​11973) (Milos Djermanovic)
  • d961438 Docs: Fix Incorrect Documentation (#​12045) (Michael Miceli)
  • 887d08c Sponsors: Sync README with website (ESLint Jenkins)
  • d90183f Docs: add a case to func-names (#​12038) (Chiawen Chen)
  • 8a5b62d Docs: no use eslint.linter in code example (#​12037) (薛定谔的猫)
  • 5831767 Update: report location of func-names (fixes #​12022) (#​12028) (Pig Fang)

v6.1.0

Compare Source

v6.0.1

Compare Source

v6.0.0

Compare Source

airbnb/javascript

v18.0.1

Compare Source

v18.0.0

Compare Source

prettier/eslint-config-prettier

v6.4.0

Compare Source

  • Added: [unicorn/no-nested-ternary]. Thanks to Yang Mingshan
    (@​yangmingshan)!

v6.3.0

Compare Source

  • Added: [@​typescript-eslint/brace-style]. Thanks to Masafumi Koba
    (@​ybiquitous)!

v6.2.0

Compare Source

  • Added: [@​typescript-eslint/quotes] (as a [special
    rule][@​typescript-eslint/quotes-special]). Thanks to Masafumi Koba
    (@​ybiquitous)!

v6.1.0

Compare Source

  • Added: [function-call-argument-newline] (new in ESLint 6.2.0). Thanks to
    Masafumi Koba (@​ybiquitous)!

v6.0.0

Compare Source

  • Changed: The CLI helper tool now considers [no-confusing-arrow] to conflict
    if you use the default value of its allowParens option. The default was
    changed to true in ESLint 6, which conflicts with Prettier.

    If the CLI helper tool gives you errors about this after upgrading, the
    solution is to change this:

    {
      "rules": {
        "no-confusing-arrow": ["error"]
      }
    }

    Into this:

    {
      "rules": {
        "no-confusing-arrow": ["error", { "allowParens": false }]
      }
    }

    The latter works in both ESLint 6 as well as in ESLint 5 and older.

  • Improved: eslint --print-config usage instructions. The CLI tool help
    text as well as the documentation has been updated to suggest commands that
    work in ESLint 6.0 as well as in ESLint 5 and older. (Instead of eslint --print-config ., use eslint --print-config path/to/main.js.)

v5.1.0

Compare Source

  • Added: [react/jsx-curly-newline]. Thanks to Masafumi Koba (@​ybiquitous)!

v5.0.0

Compare Source

  • Removed: [react/self-closing-comp]. This rule was added in v4.1.0 not
    because it conflicted with Prettier but because it was unnecessary when
    using Prettier. However, in v1.18.0 [Prettier stopped converting empty
    elements to self-closing elements][prettier-self-closing]. So the rule is
    not unnecessary anymore.

    If you use Prettier v1.17.1 or older you should be able t


Renovate configuration

📅 Schedule: "every weekend" in timezone Asia/Singapore.

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot. View repository job log here.

@netlify
Copy link

netlify bot commented Jun 15, 2019

Deploy preview for nusmods ready!

Built with commit ebf8281

https://deploy-preview-1851--nusmods.netlify.com

@renovate renovate bot force-pushed the renovate/major-eslint branch 14 times, most recently from 3a3bab7 to b1dd258 Compare June 22, 2019 03:16
@renovate renovate bot changed the title Update dependency eslint-config-prettier to v5 Update eslint (major) Jun 22, 2019
@taneliang
Copy link
Member

Failing test caused by typescript-eslint/typescript-eslint#639. We'll need to wait for @typescript-eslint/parser >1.10.2 to be released before we can work on this PR.

@renovate renovate bot force-pushed the renovate/major-eslint branch 11 times, most recently from 1282330 to 243e07a Compare June 28, 2019 23:59
@renovate renovate bot force-pushed the renovate/major-eslint branch 13 times, most recently from 564423e to 93eca0d Compare October 7, 2019 06:44
- Disabled some false positive and unwanted rules
- Downgraded some rules pending fixes
@renovate
Copy link
Contributor Author

renovate bot commented Oct 11, 2019

PR has been edited

👷 This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

Copy link
Member

@li-kai li-kai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🙇 thanks for all the work that went into this!

// @ts-ignore Not technically safe, since some ReactNodes are also functions, but this is safe enough
return children(isOnline);
// Not technically safe, since some ReactNodes are also functions, but this is safe enough
return (children as Function)(isOnline);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting, typescript still won't infer children as functions here?

@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #1851 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1851      +/-   ##
==========================================
- Coverage   45.74%   45.73%   -0.02%     
==========================================
  Files         238      238              
  Lines        4945     4944       -1     
  Branches     1159     1159              
==========================================
- Hits         2262     2261       -1     
  Misses       2683     2683
Impacted Files Coverage Δ
src/storage/persistReducer.ts 0% <0%> (ø) ⬆️
src/utils/insertScript.ts 100% <0%> (ø) ⬆️
src/actions/export.ts 12% <0%> (ø) ⬆️
src/bootstrapping/configure-store.ts 0% <0%> (ø) ⬆️
src/middlewares/requests-middleware.ts 91.66% <0%> (ø) ⬆️
src/reducers/index.ts 100% <0%> (ø) ⬆️
src/apis/export.ts 14.28% <0%> (ø) ⬆️
src/views/components/LinkModuleCodes.tsx 100% <0%> (ø) ⬆️
src/utils/react.tsx 78.37% <0%> (ø) ⬆️
src/utils/HistoryDebouncer.ts 70.58% <0%> (ø) ⬆️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 169b1fd...ebf8281. Read the comment docs.

@ZhangYiJiang ZhangYiJiang merged commit ca82302 into master Oct 11, 2019
@ZhangYiJiang ZhangYiJiang deleted the renovate/major-eslint branch October 11, 2019 17:46
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.

None yet

4 participants