Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Split out formatting rules into a separate package #3592

Closed
adidahiya opened this issue Dec 18, 2017 · 4 comments
Closed

Split out formatting rules into a separate package #3592

adidahiya opened this issue Dec 18, 2017 · 4 comments
Labels
Formatting rule Relates to a rule which enforces code formatting and likely overlaps with prettier 🌹 R.I.P. 🌹 Status: In Discussion Type: Breaking Change

Comments

@adidahiya
Copy link
Contributor

Following up on my comments here: #3395 (comment)

I'd like to separate these rules and disable them in the tslint:recommended configuration so that it's easier to use prettier with tslint. Prettier does a great job of formatting and I'd rather invest time & energy in other tslint rules that are not formatting-related.

@ajafff
Copy link
Contributor

ajafff commented Dec 19, 2017

That means you want to publish a separate npm package like @tslint/formatting? Are these rules maintained in a separate repo or is this one converted to a monorepo?

Btw. if you want to publish packages with @tslint scope on npm: I accidentally registered that org and would hand it over to you.

@adidahiya
Copy link
Contributor Author

I'd like to convert this to a monorepo; I think tracking all issues together in one repo is a good workflow we should retain.

Thanks for the offer about the @tslint scope -- I haven't decided if we should use scoped packages or use names like tslint-core, tslint-formatting, etc... but it's certainly worth considering (we'll avoid conflicts if we use the scope).

@JoshuaKGoldberg
Copy link
Contributor

💀 It's time! 💀

TSLint is being deprecated and no longer accepting pull requests for major new changes or features. See #4534. 😱

If you'd like to see this change implemented, you have two choices:

  • Recommended: Check if this is available in ESLint + typescript-eslint
  • Not Recommended: Fork TSLint locally 🤷‍♂️

👋 It was a pleasure open sourcing with you!

If you believe this message was posted here in error, please comment so we can re-open the issue!

@JoshuaKGoldberg
Copy link
Contributor

🤖 Beep boop! 👉 TSLint is deprecated 👈 (#4534) and you should switch to typescript-eslint! 🤖

🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋

@palantir palantir locked and limited conversation to collaborators Mar 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Formatting rule Relates to a rule which enforces code formatting and likely overlaps with prettier 🌹 R.I.P. 🌹 Status: In Discussion Type: Breaking Change
Projects
None yet
Development

No branches or pull requests

3 participants