Skip to content

Commit

Permalink
feat: Revamp (#325)
Browse files Browse the repository at this point in the history
* feat: Update semver range for peer dependencies

* feat: Add rules for possible problems

* chore: remove these file temporarily

* chore(deps): Upgrade ESLint to v8.5.0

* fix(possible-problems): Set this to error instead of warn

* feat(Suggestions): Add Suggestions rules

* feat(Layout-and-Formatting): Add Layout and Formatting rules

* feat: Add these files back

TypeScript support revamp soon

* fix: Use strip-json-comments to strip comments in .eslintrc.json

* feat(typescript): TypeScript version!

* fix(typescript): Better extensionRules handler

* fix(base): Add 'no-shadow' rule

* revert: "fix(typescript): Better extensionRules handler"

This reverts commit 82fd737.

* fix(typescript): The REAL Better infer handler

* chore(typescript): Add a comment there

* chore(typescript): Fix typo

* fix(typescript): Handle inferredButUndefined

* fix(typescript): Remove extensionRules that doesn't exists in baseRules

* fix(typescript): Don't remove extensionRules that is not inferred

* fix: Always suggest "return await" for better stack trace information

This will set "@typescript-eslint/return-await" rule to warn
And rule "no-return-await" will be turned off in the base rules.

It is done because for better stack trace information as well as for consistent error handling

REFS:
https://typescript-eslint.io/rules/return-await
typescript-eslint/typescript-eslint#994 (comment)
eslint/eslint#12246
https://eslint.org/docs/rules/no-return-await

* chore(base): Add todo here

* fix: Do some TODOs

* chore(typescript): remove this todo

* feat(typescript): Add node version

* fix(quotes): Don't allowTemplateLiterals without template

* feat(main): Reconsider the severity of the rules

* fix(typescript): naming-convention

* feat(typescript): Disable no-non-null-assertion

* fix(typescript): Static methods after normal method

* fix(typescript): Static fields after normal fields

* fix(main): Set max-len to 180

* fix: set @typescript-eslint/no-non-null-asserted-optional-chain to warn and remove no-param-reassign

* fix(main): Set max-depth to 5 and remove default-case rule

* fix(main): Remove no-return-assign

* fix(main): Remove consistent-return

* fix(typescript): Set no-misused-promises to warn and set checksVoidReturn option to false

* fix(typescript): Set no-unsafe-member-access severity to warn

* fix(typescript): Allow constructor to have public accessibility modifier

* fix(typescript): Fix naming-convention options

I prefer it like this.
To be honest, I'm not sure, because there's probably some thing that needs to be named in snake_case or UPPER_CASE in the future, but I'll add it to the exception list later of course

* fix(typescript): Remove no-confusing-void-expression
  • Loading branch information
Hazmi35 authored Jan 15, 2022
1 parent 31da027 commit e6de833
Show file tree
Hide file tree
Showing 7 changed files with 436 additions and 559 deletions.
Loading

0 comments on commit e6de833

Please sign in to comment.