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.

Sign up for GitHub

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

Jump to bottom

Enable noImplicitAny on the tsconfig #880

Closed
JustinBeckwith opened this issue Dec 2, 2017 · 4 comments
Closed

Enable noImplicitAny on the tsconfig #880

JustinBeckwith opened this issue Dec 2, 2017 · 4 comments
Assignees
@JustinBeckwith

Comments

@JustinBeckwith
Copy link
Contributor

We currently have a ton of any types floating around after the TypeScript conversion. We should get those cleaned up, and enable noImplictAny on the repo.
This was referenced Dec 2, 2017
Closed
Closed
@iamclaytonray
Copy link

iamclaytonray commented Jan 19, 2018 •
edited
Loading

@JustinBeckwith - I can take care of this, along with #881 and #882 if it hasn't been already. (If it's up for grabs)

@JustinBeckwith
Copy link
Contributor Author

That would be awesome!

@iamclaytonray
Copy link

Awesome, I'll start working on it in the next couple of days. Thanks 😄

@iamclaytonray
Copy link

Hey @JustinBeckwith

Just wanted to let you know that I am working on this in my spare time. I kind of got slammed with work so unfortunately, I had to put this on the back-burner. Completely understand if someone else wants to take this, otherwise, I'll have a working PR eventually. Sorry 😞

@JustinBeckwith JustinBeckwith mentioned this issue Mar 8, 2018
Merged
@JustinBeckwith JustinBeckwith self-assigned this Mar 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JustinBeckwith JustinBeckwith

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JustinBeckwith @iamclaytonray