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.

Sign up for GitHub

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

Jump to bottom

Upgrade ESLint and all plugis & revalidate the config and ALL of the rules #15267

Closed
wants to merge 29 commits into from
Closed

Upgrade ESLint and all plugis & revalidate the config and ALL of the rules #15267

wants to merge 29 commits into from

Conversation

peter-sanderson
Copy link
Contributor

@peter-sanderson peter-sanderson commented Nov 6, 2024 •
edited
Loading

Todos:

  • @typescript-eslint/triple-slash-reference remove off

This was referenced Nov 6, 2024
Closed
Closed
@peter-sanderson peter-sanderson changed the title Eslint upgrade3 Upgrade ESLint and all plugis & revalidate the config and ALL of the rules Nov 6, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024

🚀 Expo preview is ready!

  • Project → trezor-suite-preview
  • Platforms → android, ios
  • Scheme → trezorsuitelite
  • Runtime Version → 9
  • More info

Learn more about 𝝠 Expo Github Action

@peter-sanderson
chore: more ESLint fixies
0019544
@peter-sanderson
chore: get rid of '@typescript-eslint/no-unused-vars': 'off', and enf…
c8aa119 
…orce it everywhere
@peter-sanderson
fix: wip
70da002
@peter-sanderson
fix: local-rules/no-override-ds-component were not riggered correcly …
b1456af 
…due to incorrect configuration
@peter-sanderson
chore: enable no-constant-binary-expression as it becames recommanded…
8a23a79 
… default in new ESLint
@peter-sanderson
chore: get rid of '@typescript-eslint/no-unused-vars': 'off', and enf…
fe888c1 
…orce it everywhere
@peter-sanderson
chore: wip
812fb29
@peter-sanderson
chore: enable no-extra-boolean-cast as it becames recomended
c162608
@peter-sanderson
chore: wip
82a9290
@peter-sanderson
chore: add no-async-promise-executor as it becames recommanded
7c73c09
@peter-sanderson
chore: enable import/order rule for whole codebase
d1748b5
@peter-sanderson
chore: enable import/order rule for whole codebase - manual fixies
c4ab992
@peter-sanderson
chore: enable import/order rule for whole codebase - prettier fix
c56c7d7
@peter-sanderson
chore: remove all import/order offs
0d37421
@peter-sanderson
chore: ignore some rules in packages
2634229
@peter-sanderson
chore: no-control-regex as it becames recommanded
74df31e
@peter-sanderson
chore: wip
b3a0ee3
@peter-sanderson
Copy link
Contributor Author

=> #15285

@peter-sanderson
Copy link
Contributor Author

=> #15287

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martykan martykan Awaiting requested review from martykan martykan is a code owner

@szymonlesisz szymonlesisz Awaiting requested review from szymonlesisz szymonlesisz is a code owner

@karliatto karliatto Awaiting requested review from karliatto karliatto is a code owner

@mroz22 mroz22 Awaiting requested review from mroz22 mroz22 is a code owner

@marekrjpolak marekrjpolak Awaiting requested review from marekrjpolak marekrjpolak is a code owner

@komret komret Awaiting requested review from komret komret is a code owner

@tomasklim tomasklim Awaiting requested review from tomasklim tomasklim is a code owner

@matejkriz matejkriz Awaiting requested review from matejkriz matejkriz is a code owner

@jvaclavik jvaclavik Awaiting requested review from jvaclavik

@seibei-iguchi seibei-iguchi Awaiting requested review from seibei-iguchi

@Nodonisko Nodonisko Awaiting requested review from Nodonisko Nodonisko is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@peter-sanderson