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 @typescript-eslint/eslint-plugin requirement from ^1.7.0 to ^2.0.0 #40

Conversation

dependabot-preview[bot]
Copy link
Contributor

Updates the requirements on @typescript-eslint/eslint-plugin to permit the latest version.

Release notes

Sourced from @typescript-eslint/eslint-plugin's releases.

v2.0.0

2.0.0 (2019-08-13)

BREAKING CHANGES

  • Node 6 is no longer supported.
  • parser / typescript-estree:
    • When project is specified within parserOptions, we will now hard fail when parsing files that are not included within the provided tsconfig(s).
    • We discovered that this was a common performance pitfall, and could increase lint times by huge amounts.
    • To handle this, there are a few possible solutions:
      1. Improve the includes field within your tsconfig(s) so that all the files you want to lint are included.
      2. Create a new tsconfig.eslint.json which you pass into parserOptions.project, which includes all of the files you want to lint, e.g.:
        {
          // extend your base config so you don't have to redefine your compilerOptions
          "extends": "./tsconfig.json",
          "include": [
            "src/**/*.ts",
            "test/**/*.ts",
            "typings/**/*.ts"
            // etc
          ],
          // IF (and only if) you have a mixed JS/TS codebase - you should also turn on JS support
          "compilerOptions": {
            "allowJs": true,
            "checkJs": true
          }
        }
        If you are using non standard file extensions (i.e. .vue files), you should add the following config to your .eslintrc file:
        {
          "parserOptions": {
            "extraFileExtensions": [".vue"]
          }
        }
  • eslint-plugin:
    • Removed hardcoded checks which prevented some rules from running on non-TypeScript files (i.e. files that are not *.ts/*.tsx).
      • Some users with mixed TS/JS codebases may now see some TypeScript-specific rules being reported against JS files.
      • Please use ESLint's "overrides" config to select which files to apply rules to.
    • Removed deprecated rule prefer-interface. This rule was replaced by consistent-type-definitions.
    • Removed deprecated rule no-triple-slash-reference. This rule was replaced by triple-slash-reference
    • Merged both no-angle-bracket-type-assertion and no-object-literal-type-assertion into one rule - consistent-type-assertions.
    • explicit-function-return-type no longer treats export default () => {} as an expression for the purposes of the allowExpressions option.
    • Changed a number of the default rule configs:
      • explicit-function-return-type - allowTypedFunctionExpressions and allowHigherOrderFunctions are now both true by default.
      • no-inferrable-types - ignoreParameters and ignoreProperties are now both false by default.
      • no-this-alias - allowDestructuring is now true by default.
... (truncated)
Changelog

Sourced from @typescript-eslint/eslint-plugin's changelog.

2.0.0 (2019-08-13)

Bug Fixes

  • eslint-plugin: [efrt] flag default export w/allowExpressions (#831) (ebbcc01)

  • eslint-plugin: [no-explicit-any] Fix ignoreRestArgs for interfaces (#777) (22e9ae5)

  • eslint-plugin: [no-useless-constructor] handle bodyless constructor (#685) (55e788c)

  • eslint-plugin: [prefer-readonly] TypeError when having comp… (#761) (211b1b5)

  • eslint-plugin: [typedef] support "for..in", "for..of" (#787) (39e41b5)

  • eslint-plugin: [typedef] support default value for parameter (#785) (84916e6)

  • feat(eslint-plugin)!: recommended-requiring-type-checking config (#846) (d3470c9), closes #846

  • feat(eslint-plugin)!: change recommended config (#729) (428567d), closes #729

  • feat(typescript-estree)!: throw error on file not in project when project set (#760) (3777b77), closes #760

  • feat(eslint-plugin)!: add rule consistent-type-assertions (#731) (92e98de), closes #731

  • feat(eslint-plugin)!: [array-type] rework options (#654) (1389393), closes #654 #635

Features

  • explicitly support eslint v6 (#645) (34a7cf6)
  • eslint-plugin: [interface-name-prefix, class-name-casing] Add allowUnderscorePrefix option to support private declarations (#790) (0c4f474)
  • eslint-plugin: [no-var-requires] report on foo(require('')) (#725) (b2ca20d), closes #665
  • eslint-plugin: [promise-function-async] make allowAny default true (#733) (590ca50)
  • eslint-plugin: [strict-boolean-expressions] add ignoreRhs option (#691) (fd6be42)
  • eslint-plugin: add support for object props in CallExpressions (#728) (8141f01)
  • eslint-plugin: added new rule typedef (#581) (35cc99b)
  • eslint-plugin: added new rule use-default-type-parameter (#562) (2b942ba)
  • eslint-plugin: move opinionated rules between configs (#595) (4893aec)
  • eslint-plugin: remove deprecated rules (#739) (e32c7ad)

BREAKING CHANGES

  • removed some rules from recommended config
  • recommended config changes are considered breaking
  • by default we will now throw when a file is not in the project provided
  • Merges both no-angle-bracket-type-assertion and no-object-literal-type-assertion into one rule
  • eslint-plugin: both 'eslint-recommended' and 'recommended' have changed.
  • eslint-plugin: removing rules
  • changes config structure
type ArrayOption = 'array' | 'generic' | 'array-simple';
type Options = [
  {
    // default case for all arrays
    default: ArrayOption,
</tr></table> ... (truncated)
Commits
  • 05ba268 chore: publish v2.0.0
  • 0c4f474 feat(eslint-plugin): [interface-name-prefix, class-name-casing] Add allowUnde...
  • d3470c9 feat(eslint-plugin)!: recommended-requiring-type-checking config (#846)
  • 90b36dd docs(eslint-plugin): update ROADMAP.md (#844)
  • 428567d feat(eslint-plugin)!: change recommended config (#729)
  • de6cc1d docs(eslint-plugin): [no-useless-constructor] add example setup (#837)
  • ebbcc01 fix(eslint-plugin): [efrt] flag default export w/allowExpressions (#831)
  • 42b3013 chore: misc package.json updates related to v2 (#832)
  • 39e41b5 fix(eslint-plugin): [typedef] support "for..in", "for..of" (#787)
  • 84916e6 fix(eslint-plugin): [typedef] support default value for parameter (#785)
  • Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot dashboard:

  • Update frequency (including time of day and day of week)
  • Pull request limits (per update run and/or open at any time)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

Finally, you can contact us by mentioning @dependabot.

@dependabot-preview dependabot-preview bot added the type:dependencies Pull requests that update a dependency file label Aug 22, 2019
@sseliverstov
Copy link
Collaborator

@dependabot merge

@dependabot-preview
Copy link
Contributor Author

Dependabot tried to merge this PR, but received the following error from GitHub:

At least 1 approving review is required by reviewers with write access.

@sseliverstov sseliverstov self-requested a review August 22, 2019 11:43
@sseliverstov
Copy link
Collaborator

@dependabot merge

@dependabot-preview dependabot-preview bot merged commit dd7842f into master Aug 22, 2019
@dependabot-preview dependabot-preview bot deleted the dependabot/npm_and_yarn/typescript-eslint/eslint-plugin-tw-2.0.0 branch August 22, 2019 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant