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

new rule: pascalCaseClassNames #4175

Closed
jpike88 opened this issue Jan 24, 2023 · 3 comments
Closed

new rule: pascalCaseClassNames #4175

jpike88 opened this issue Jan 24, 2023 · 3 comments
Labels
A-Linter Area: linter task A task, an action that needs to be performed

Comments

@jpike88
Copy link

jpike88 commented Jan 24, 2023

Description

Class names usually adopt the pascal case convention for readability, and standing them out from variables, functions.

This would be invalid:

class thisIsAClass { }

This would be valid:

class ThisIsAClass { }
@jpike88 jpike88 added the task A task, an action that needs to be performed label Jan 24, 2023
@Conaclos Conaclos added the A-Linter Area: linter label Feb 22, 2023
@unvalley
Copy link
Contributor

I'll work on this.

@ematipico
Copy link
Contributor

Actually, there's an interesting proposal from @Conaclos #4282 I really like.

@jpike88 when you have an idea, please use GitHub discussions instead. We use issues for bugs, technical tasks and tracking tasks are already planned/discussed.

@unvalley please hold off on this one, I'd suggest check umbrella issues around ESLint compatibility, accessibility or react

@jpike88
Copy link
Author

jpike88 commented Apr 5, 2023

closing in favor of #4282

@jpike88 jpike88 closed this as completed Apr 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Linter Area: linter task A task, an action that needs to be performed
Projects
None yet
Development

No branches or pull requests

4 participants