Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend naming ruleset #302

Closed
schalkms opened this issue Aug 13, 2017 · 6 comments
Closed

Extend naming ruleset #302

schalkms opened this issue Aug 13, 2017 · 6 comments

Comments

@schalkms
Copy link
Member

Maybe, add some more rules which detect too long or too short names.

This could be tagged as "easy".

reference file: NamingConventionViolation.kt

@arturbosch
Copy link
Member

Maybe also some words that are forbidden to use in class names like data or manager.

@Khang-NT
Copy link

It maybe more helpful if detekt has these rules for NamingConvention like Intelliji Java inspections.
screen shot 2017-08-27 at 10 40 12 am

@Mauin
Copy link
Collaborator

Mauin commented Sep 7, 2017

Should we close this issue after #374 is merged? Or should we work on some more naming rules mentioned in the IntelliJ warnings as well? Should we break those up into separate issues to make them easier to track and easier for contributors to pick up?

@vanniktech
Copy link
Contributor

I'd say close or split them up individually so people can say what they're working on.

@arturbosch
Copy link
Member

Thats a lot of rules if we want to implement them all. I also want to close this and if users want more rules, they can feature request them or just file a PR ;)

@arturbosch arturbosch modified the milestone: RC4-3 Sep 8, 2017
@lock
Copy link

lock bot commented Jun 20, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related topics.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants