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

'allow-leading-underscore' option for 'variable-name' rule #4869

Closed
ravshansbox opened this issue Oct 3, 2019 · 2 comments · Fixed by #4871
Closed

'allow-leading-underscore' option for 'variable-name' rule #4869

ravshansbox opened this issue Oct 3, 2019 · 2 comments · Fixed by #4871
Labels
Status: Accepting PRs Type: Enhancement

Comments

@ravshansbox
Copy link

The 'recommended' configuration should include 'allow-leading-underscore' option for 'variable-name' rule, because this pattern is used when skipping parameters in callback functions.
@JoshuaKGoldberg
Copy link
Contributor

Per #4811, we can take this in now.

@JoshuaKGoldberg JoshuaKGoldberg self-assigned this Oct 5, 2019
@JoshuaKGoldberg JoshuaKGoldberg mentioned this issue Oct 5, 2019
Merged
2 tasks
@JoshuaKGoldberg JoshuaKGoldberg removed their assignment Apr 11, 2020
@JoshuaKGoldberg
Copy link
Contributor

🤖 Beep boop! 👉 TSLint is deprecated 👈 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 Sep 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Status: Accepting PRs Type: Enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removed annoying formatting rules from recommended ruleset
2 participants
@ravshansbox @JoshuaKGoldberg