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

add 3 lints for always specify types #1153

Closed
wants to merge 2 commits into from

Conversation

a14n
Copy link
Contributor

@a14n a14n commented Sep 4, 2018

@pq
Copy link
Member

pq commented Sep 11, 2018

Sorry to let this one languish! I admit I'm a little reluctant to see such a proliferation of lints (if it's avoidable). Besides the argument for completeness, do we have the sense that this level of granularity would benefit a lot of folks?

@a14n
Copy link
Contributor Author

a14n commented Sep 12, 2018

I've just followed #1144 (comment) . I don't have a strong opinion on that.
If you prefer a single lint then #1144 is the PR to consider. Let me know what's you decide with @bwilkerson

@a14n a14n force-pushed the always_specify_types-split branch from 36b8cf3 to 2eba7b7 Compare September 21, 2018 06:49
@a14n a14n force-pushed the always_specify_types-split branch from 2eba7b7 to bdeed1f Compare September 21, 2018 06:50
@mit-mit
Copy link
Member

mit-mit commented Aug 20, 2019

Hi @a14n this appears stale; can we close it?

@a14n a14n closed this Aug 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a lint to require types on generic methods
4 participants