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

feat: Enforce noImplictAny #275

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feat: Enforce noImplictAny #275

wants to merge 2 commits into from

Conversation

ExE-Boss
Copy link
Contributor

@ExE-Boss ExE-Boss commented Feb 27, 2020 •
edited
Loading

This enforces that packages have noImplicitAny set to true.

Note that there’s no package in DefinitelyTyped with strict set to false.

Depends on:

sandersn
sandersn suggested changes Feb 27, 2020
View reviewed changes
Copy link
Member

@sandersn sandersn left a comment

Choose a reason for hiding this comment

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

I like the idea, but we could probably be stricter about strict too.

src/checks.ts Outdated Show resolved Hide resolved
@ExE-Boss ExE-Boss mentioned this pull request Mar 2, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sandersn sandersn sandersn requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ExE-Boss @sandersn