Skip to content

Chore: Replace tslint with eslint and improve typing of app#72

Merged
murtaza98 merged 6 commits intomasterfrom
chore/replace-tslint
Dec 1, 2021
Merged

Chore: Replace tslint with eslint and improve typing of app#72
murtaza98 merged 6 commits intomasterfrom
chore/replace-tslint

Conversation

@KevLehman
Copy link
Copy Markdown
Member

  • Replace TSLint (since it's deprecated)
  • Added ESLint support for app
  • Added Prettier support for app
  • Fixed ESLint warnings due to use of and added new type declarations

@KevLehman KevLehman requested a review from murtaza98 November 29, 2021 16:38
@murtaza98
Copy link
Copy Markdown
Contributor

Hi, @KevLehman There are a couple of conflicts introduced here due to recent merges...Could you please take a look at them?

@KevLehman
Copy link
Copy Markdown
Member Author

Updated. Please check that my resolutions are the right ones 👀

KevLehman and others added 2 commits November 30, 2021 11:08
- Fix unused variable issues
- Avoid using console.error(e)
@murtaza98
Copy link
Copy Markdown
Contributor

Hey, @KevLehman I've included my suggestions within this PR - #73. Please review and merge them here when you get a chance 😄

Thanks for this initiative 💙

@KevLehman KevLehman requested a review from murtaza98 November 30, 2021 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants