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

Tslint updater #274

Merged
merged 8 commits into from
Feb 25, 2020
Merged

Tslint updater #274

merged 8 commits into from
Feb 25, 2020

Conversation

gabritto
Copy link
Member

This PR creates a script that runs TSLint for DT packages, taking as configuration dt.json and ignoring the existing tslint.json for the package. The output is a new tslint.json for the packages, that only adds exemptions for rules that actually produced lint failures.

@gabritto gabritto requested a review from sandersn February 24, 2020 19:17
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.

Initial comment; I'll review the actual code tomorrow.

src/updateConfig.ts Show resolved Hide resolved
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 think it can go in as-is, although I had a couple of tiny style suggestions.

src/updateConfig.ts Outdated Show resolved Hide resolved
src/updateConfig.ts Outdated Show resolved Hide resolved
src/updateConfig.ts Outdated Show resolved Hide resolved
@gabritto gabritto merged commit 70e472c into master Feb 25, 2020
@gabritto gabritto deleted the tslintUpdater branch February 25, 2020 18:27
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