Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Commit

Permalink
Point ExpectRule users to tsd in README
Browse files Browse the repository at this point in the history
  • Loading branch information
sandersn committed Sep 5, 2019
1 parent 4a9823c commit 18e1c00
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@ It will install `typescript` and `tslint` for you, so this is the only tool you

Lint rules new to dtslint are documented in the [docs](docs) directory.

# Just looking for ExpectType and ExpectError?

[Use tsd instead](https://github.com/SamVerschueren/tsd).

# Setup

If you are working on DefinitelyTyped, read the [DefinitelyTyped README](https://github.com/DefinitelyTyped/DefinitelyTyped#readme).
Expand Down

4 comments on commit 18e1c00

@OliverJAsh
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @sanderssn. I'm curious why you recommend this. Is DefinitelyTyped moving away from using the expect lint rule?

@sandersn
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not right now. We discussed it and decided that it doesn’t integrate well enough with our current tslint rules [1]. The reason is that I don’t believe people want the non-expect tslint rules, especially the DT-structure ones. And I don’t think they want the multi-version testing either. I could be wrong about those.

[1] However, I’ll probably re-evaluate when we decide to switch to eslint.

@orta
Copy link
Contributor

@orta orta commented on 18e1c00 Sep 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yeah, great - adding this message was on my TODO also - thanks

@SamVerschueren
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the referral :)! 🎉

Please sign in to comment.