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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps-dev): migrate dtslint to @definitelytyped/dtslint #2295

Merged
merged 1 commit into from Feb 2, 2022

Conversation

mastermatt
Copy link
Member

Same tool, DefinitelyTypes simply moved the project into their monorepo.

I did attempt to move away from dtslint all together, in favor of ESLint, which is preferred for proper Typescript projects. However, ESLint still lacks a lot of functionality for only declaration files and tests, like Nock has. So dtslint is still the way to go (until we migrate Nock to TS 馃槃 ).

Same tool, DefinitelyTypes simply moved the project into their monorepo.

I did attempt to move away from dtslint all together, in favor of ESLint, which is preferred for proper Typescript projects. However, ESLint still lacks a lot of functionality for only declaration files and tests, like Nock has. So dtslint is still the way to go (until we migrate Nock to TS :) ).
@mastermatt mastermatt added the dependencies Pull requests that update a dependency file label Feb 2, 2022
@mastermatt mastermatt requested a review from a team February 2, 2022 15:21
@gr2m gr2m merged commit 51c9cf5 into main Feb 2, 2022
@gr2m gr2m deleted the migrate-dtslint branch February 2, 2022 21:13
@github-actions
Copy link

github-actions bot commented Feb 3, 2022

馃帀 This PR is included in version 13.2.3 馃帀

The release is available on:

Your semantic-release bot 馃摝馃殌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants