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

build: migrate to TypeScript 4 and remove TSLint #765

Merged
merged 6 commits into from
Oct 26, 2020

Conversation

xballoy
Copy link
Contributor

@xballoy xballoy commented Oct 22, 2020

I migrated TypeScript to the latest version but our ESLint / TSLint settings were not compatible (showing errors on valid TypeScript 4 code).

I removed TSLint which is deprecated and used eslint-config-airbnb-typescript instead so we have the same rules for both JavaScript and TypeScript.
However as we didn't use the same rules on TypeScript and JavaScript files I had to disable the following rules:

❓ Should I create issues to fix them?

I also added the following in the tsconfig to remove the * as in the TypeScript files

"esModuleInterop": true,
"allowSyntheticDefaultImports": true

Fixes #736

@xballoy
Copy link
Contributor Author

xballoy commented Oct 22, 2020

Waiting for #645 to complete to fix master

@xballoy xballoy force-pushed the feature/typescript-migration branch from a143e6f to 1cd639b Compare October 23, 2020 14:13
@xballoy xballoy marked this pull request as ready for review October 23, 2020 14:14
youf-olivier
youf-olivier previously approved these changes Oct 23, 2020
@youf-olivier
Copy link
Contributor

I migrated TypeScript to the latest version but our ESLint / TSLint settings were not compatible (showing errors on valid TypeScript 4 code).

I removed TSLint which is deprecated and used eslint-config-airbnb-typescript instead so we have the same rules for both JavaScript and TypeScript.
However as we didn't use the same rules on TypeScript and JavaScript files I had to disable the following rules:

❓ Should I create issues to fix them?

I also added the following in the tsconfig to remove the * as in the TypeScript files

"esModuleInterop": true,
"allowSyntheticDefaultImports": true

Fixes #736

Yes it could be a good idea to open issue to discuss about the lint rules.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

95.3% 95.3% Coverage
0.0% 0.0% Duplication

@mergify mergify bot merged commit 3c5fe86 into master Oct 26, 2020
@delete-merged-branch delete-merged-branch bot deleted the feature/typescript-migration branch October 26, 2020 08:07
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.

Migrate to TypeScript 4
3 participants