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

Add clarity about TSLint plugins #386

Merged
merged 3 commits into from Jun 17, 2023

Conversation

AlessioGr
Copy link
Contributor

@AlessioGr AlessioGr commented Jun 16, 2023

Fixes #385

I've also named those tools in the main, non-FAQ page which should make the FAQ easier to find. I, personally, tried to ctrl+F search for the names of those plugins on that page and didn't find anything. This will make it easier and clarify, that you can find information about those tools on the FAQ page.

@vercel
Copy link

vercel bot commented Jun 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
goldblog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2023 1:01pm

Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

LGTM! Just dubious on the tl;dr - otherwise very good stuff! Thanks 🙌


The same applies to [tslint-config-prettier](https://github.com/prettier/tslint-config-prettier) and [tslint-plugin-prettier](https://github.com/prettier/tslint-plugin-prettier). Very often, those are still recommended if you're using TypeScript (and you'll hear stuff along the lines of "use this for TypeScript"). This is not true - those two tools work together with **TSLint**, not plain, standalone TypeScript.

Since TSLint is deprecated and not recommended anymore, these two plugins shouldn't be used anymore as well, since they're specifically made for TSLint. You don't need them for TypeScript.
Copy link
Owner

Choose a reason for hiding this comment

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

[Praise] Great! Excellent section 😄

Co-authored-by: Josh Goldberg ✨ <git@joshuakgoldberg.com>
Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

💯, thanks a million @AlessioGr!

@JoshuaKGoldberg JoshuaKGoldberg merged commit 7c443ff into JoshuaKGoldberg:main Jun 17, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a mention of tslint-config-prettier to Configuring ESLint, Prettier, and TypeScript Together
2 participants