fix: linter - Githubissues

symbol / mobile-wallet

Symbol Mobile Wallet

Apache License 2.0

8 stars 14 forks source link

fix: linter #266

Closed OlegMakarenko closed 2 years ago

OlegMakarenko commented 2 years ago

Added Eslint and Prettier.
Added Husky pre-commit hook.
Added rules:
- react/recommended.
- prettier/recommended.
- sort-imports: error.
- no-unused-vars: error.
Applied linter.
Manually removed unused variables and properties (needs to be tested).

lgtm-com[bot] commented 2 years ago

This pull request introduces 5 alerts and fixes 155 when merging 6cbb54364e6a01c933c342e97256f9ec08851338 into f08856010e21ba74abf338ee6d3076141fcb3056 - view on LGTM.com

new alerts:

5 for Unused or undefined state property

fixed alerts:

150 for Unused variable, import, function or class
3 for Unused or undefined state property
2 for Direct state mutation

lgtm-com[bot] commented 2 years ago

This pull request introduces 5 alerts and fixes 156 when merging 2544ae8ee4670e1dfe12e1e33db5f623ddf05992 into f08856010e21ba74abf338ee6d3076141fcb3056 - view on LGTM.com

new alerts:

5 for Unused or undefined state property

fixed alerts:

150 for Unused variable, import, function or class
3 for Unused or undefined state property
2 for Direct state mutation
1 for Unreachable statement

sonarcloud[bot] commented 2 years ago

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
1 Security Hotspot
35 Code Smells

No Coverage information
9.2% Duplication

lgtm-com[bot] commented 2 years ago

This pull request introduces 5 alerts and fixes 156 when merging 51c36c6b627c14ff368f2f7797c068803b82fb18 into f08856010e21ba74abf338ee6d3076141fcb3056 - view on LGTM.com

new alerts:

5 for Unused or undefined state property

fixed alerts:

150 for Unused variable, import, function or class
3 for Unused or undefined state property
2 for Direct state mutation
1 for Unreachable statement