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

Check number of empty lines between methods in classes #3517

Closed
Franziskus1988 opened this issue Nov 28, 2017 · 7 comments
Closed

Check number of empty lines between methods in classes #3517

Franziskus1988 opened this issue Nov 28, 2017 · 7 comments
Labels
Formatting rule Relates to a rule which enforces code formatting and likely overlaps with prettier 🌹 R.I.P. 🌹 Resolution: Declined Type: Rule Suggestion

Comments

@Franziskus1988
Copy link

Feature request

  • TSLint version: 5.8.0

Proposal

In our company we have style guides regarding the number of empty lines between methods in classes. Would it be possible to add a rule to check this style?

@ajafff
Copy link
Contributor

ajafff commented Nov 28, 2017

Could be implemented as part of #3139

@adidahiya
Copy link
Contributor

This has some conceptual overlap with no-consecutive-blank-lines. If we're creating a newline rule then we should fold that functionality into the new rule as well.

@vegarringdal
Copy link

Anyone working on this one ?
Would be really nice to have.

@JoshuaKGoldberg
Copy link
Contributor

Related: #975, #3777.

If there's a need for an expanded, configurable newline rule, it probably makes sense to include options for all these issues.

@JoshuaKGoldberg
Copy link
Contributor

Per #4534, closing this formatting rule request. If you really need the rule, custom rules are always an option and can be maintained outside this repo!

@JoshuaKGoldberg JoshuaKGoldberg added Formatting rule Relates to a rule which enforces code formatting and likely overlaps with prettier Resolution: Declined and removed Status: In Discussion Status: Needs Proposal labels Jul 4, 2019
@JoshuaKGoldberg
Copy link
Contributor

💀 It's time! 💀

TSLint is being deprecated and no longer accepting pull requests for new rules. See #4534. 😱

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

👋 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. 🌹 Resolution: Declined Type: Rule Suggestion
Projects
None yet
Development

No branches or pull requests

6 participants