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

check variable names #93

Open
LucaPaterlini opened this issue Jun 15, 2019 · 5 comments
Open

check variable names #93

LucaPaterlini opened this issue Jun 15, 2019 · 5 comments

Comments

@LucaPaterlini
Copy link

an option like
"-hard"
It will mark as mistake "else" and will mark as mistake all the 1 character variable aside those declared inside a for or if statement.

@tehsphinx
Copy link
Member

Can you name linters that would do this?

@LucaPaterlini
Copy link
Author

actually, there is none that I know, but I suppose a script that apply a regex looking for those is something possible to implement

@bitfield
Copy link
Contributor

Isn't this really a more general question about automated code analyzers for all Exercism languages?

@LucaPaterlini
Copy link
Author

would be really nice if get implemented for all the languages supported by Exercism :)

@tehsphinx
Copy link
Member

tehsphinx commented Jun 18, 2019

We will probably do something at some point about checking variable names. There are some naming guidelines that are enforced by linters and I would probably start with that.

Note that we are having discussions on this in the auto-mentor project: https://github.com/exercism/automated-mentoring-support/blob/master/docs/guidelines.md#approvability

@tehsphinx tehsphinx changed the title feature request check for variable names Jun 20, 2019
@tehsphinx tehsphinx changed the title check for variable names check variable names Jun 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants