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.

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

migration from tslint to eslint #58

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

andkononykhin
Copy link
Contributor

@andkononykhin andkononykhin commented Nov 29, 2021

Context:

ToDo

  • review wasm/demo/tslint-to-eslint-config.log and wasm/tests-js/tslint-to-eslint-config.log and:
    • adjust rules / code if needed
    • remove the logs
  • review wasm/demo/tsconfig.json and wasm/tests-js/tsconfig.json (both are empty for now) and adjust if needed
    some docs
  • review comments and rules in generated wasm/demo/.eslintrc.js and wasm/tests-js/.eslintrc.js and try to follow the recommendations
  • run checks, review the failures and make them fixed or disable

Signed-off-by: Andrey Kononykhin <[email protected]>
@andkononykhin andkononykhin changed the title migrates from tslint to eslint migration from tslint to eslint Nov 29, 2021
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.

1 participant