Skip to content
This repository was archived by the owner on Jan 19, 2019. It is now read-only.

New: Add rule no-any #8

Merged
merged 1 commit into from
Oct 23, 2016
Merged

New: Add rule no-any #8

merged 1 commit into from
Oct 23, 2016

Conversation

dannyfritz
Copy link
Contributor

Rule based off of TSLint's no-any rule

@dannyfritz dannyfritz mentioned this pull request Oct 13, 2016
31 tasks
@JamesHenry
Copy link
Collaborator

JamesHenry commented Oct 13, 2016

Thanks for your continued efforts, @dannyfritz!

Could we go for no-explicit-any as the name of the rule?

I think it is a better explanation of what is actually being checked, and that way it is more clearly a complement to the TypeScript compiler flag noImplicitAny. It is possible to have (implicit) any types in your codebase, even with this rule enabled, if you have not enabled noImplicitAny in your project.

Could you also briefly touch on that relationship in your docs?

@dannyfritz
Copy link
Contributor Author

Rebased and made changes. :)

@JamesHenry
Copy link
Collaborator

LGTM, thanks!

@JamesHenry JamesHenry merged commit ec19936 into bradzacher:master Oct 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants