Skip to content
This repository has been archived by the owner on Dec 5, 2024. It is now read-only.

Support typescript eslint4.15 #53

Merged
merged 2 commits into from
Mar 1, 2021
Merged

Conversation

FredrikJT
Copy link
Contributor

https://github.com/tobiipro/support-firecloud/blob/master/doc/working-with-git-pr.md :
0. Small is Best

  1. Correct
  2. Consistent
  3. Readable
  4. Share Knowledge
    -->

The format of the recommended config file has change from .json to .js when going from typescript-eslint@2 to typescript-eslint@4.
Some rules has been changed and deprecated which can be read about here: https://github.com/typescript-eslint/typescript-eslint/releases/tag/v3.0.0

…ention rule

These changes are introduced in typescript-eslint 3 and can be read about here: https://github.com/typescript-eslint/typescript-eslint/releases/tag/v3.0.0

Changes made:
- ban-ts-remove is replaced by ban-rs-comment
- The new rule naming-convention is used to re-implement the behavior of a few deprecated rules:
  - interface-name-prefix
  - class-name-casing
  - generic-type-naming
@FredrikJT FredrikJT force-pushed the support-typescript-eslint4.15 branch from fefe735 to 35890ce Compare February 26, 2021 13:58
@FredrikJT FredrikJT merged commit f5c1707 into master Mar 1, 2021
@FredrikJT FredrikJT deleted the support-typescript-eslint4.15 branch March 1, 2021 13:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

eslint-config-firecloud does not support typescript 4 which is part of Angular 10
2 participants