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

Indentation Format #15

Closed
elbrujohalcon opened this issue Jun 26, 2014 · 7 comments
Closed

Indentation Format #15

elbrujohalcon opened this issue Jun 26, 2014 · 7 comments
Labels
Milestone

Comments

@elbrujohalcon
Copy link
Member

Rule

Indent using {N spaces|tabs}

Options
  • Indentation character
  • (if using spaces) number of spaces
Implementation

Check in each line, how much more indented it is against the previous one.

@elbrujohalcon elbrujohalcon added this to the 1.0.0 - With all da rulez! milestone Jun 26, 2014
@jfacorro
Copy link
Contributor

jfacorro commented Sep 9, 2014

@elbrujohalcon Since this rule was put on hold we should move it to milestone 1.1.0

@jfacorro jfacorro modified the milestones: 1.1.0 - With MORE rulez, 1.0.0 - With all da rulez!, 1.1.1 - Hooks and/or improvements and/or research Sep 9, 2014
@essen
Copy link

essen commented Nov 21, 2014

Commenting here because we still only have the no_tabs option and that's unacceptable. Where is my option no_spaces that ensures there's no nasty little blanks in my files? I want a refund!

@jfacorro
Copy link
Contributor

@essen Do you mean having a no_spaces rule that detects trailing spaces, spaces as indentation or both? #147 is for trailing whitespace, but there's currently no issue open for a no_spaces rule. Please fell free to open one!

@essen
Copy link

essen commented Nov 21, 2014

I mean space as indentation. The good equivalent of the evil no_tabs.

Trailing whitespace (space or tab) would also be nice but it already has a separate ticket for it.

@jfacorro
Copy link
Contributor

I just created the issue #178, but since creating new rules is not one of our priorities right now, it might be a while until we actually implement it. PRs are always welcomed 😉!

@essen
Copy link

essen commented Nov 21, 2014

I'm still stuck processing RFCs and will be for the foreseeable future so don't expect much.

@elbrujohalcon
Copy link
Member Author

Since #178 is there, I'll close this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants