search

DefinitelyTyped / dt-mergebot

The bot which handles auto-merging your PRs
https://devblogs.microsoft.com/typescript/changes-to-how-we-manage-definitelytyped/
MIT License
112 stars 44 forks source link

Remove tslint #484

Closed sandersn closed 10 months ago

sandersn commented 10 months ago

And do not add checking for eslint. Not sure that's the right choice.

OTHER_FILES.txt also isn't checked anymore. Also there are some unused link constants.

sandersn commented 10 months ago

@JoshuaKGoldberg you are probably interested in this too.

JoshuaKGoldberg commented 10 months ago

👍 this is of interest to me, thanks for the ping! I don't have anything to add but I am very happy about it. 🙂

jakebailey commented 10 months ago

Or maybe we are forced to leave it, just return undefined like the other files?

sandersn commented 10 months ago

I'd rather have a PR ready to remove them from DT first, then merge both the dt-tools PR and this PR. At least, I don't think there's any point in allowing-but-not-checking them like .npmignore et al.