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

update RulesCommand to print identifiers but not examples #164

Merged
merged 1 commit into from
Oct 19, 2015

Conversation

jpsim
Copy link
Collaborator

@jpsim jpsim commented Oct 19, 2015

The previous functionality lost its usefulness IMHO because the rule examples that are also used for unit tests lost their "printability" and readability over time.

This PR greatly simplifies the rules command to just print a list of rule names and identifiers, making this a much easier way to find all the available rules:

$ swiftlint rules
Line Length Rule (line_length)
Leading Whitespace Rule (leading_whitespace)
Trailing Whitespace Rule (trailing_whitespace)
Returning Whitespace Rule (return_arrow_whitespace)
Trailing Newline Rule (trailing_newline)
Operator Function Whitespace Rule (operator_whitespace)
Force Cast Rule (force_cast)
File Line Length Rule (file_length)
Todo Rule (todo)
Colon Rule (colon)
Type Name Rule (type_name)
Variable Name Rule (variable_name)
Type body Length Rule (type_body_length)
Function Body Length Rule (function_body_length)
Nesting Rule (nesting)
Control Statement Rule (control_statement)

@jpsim jpsim force-pushed the jp-new-rules-command branch 2 times, most recently from b4faa6e to ba35fe9 Compare October 19, 2015 19:15
jpsim added a commit that referenced this pull request Oct 19, 2015
update RulesCommand to print identifiers but not examples
@jpsim jpsim merged commit 57da31b into master Oct 19, 2015
@jpsim jpsim deleted the jp-new-rules-command branch October 19, 2015 20:19
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

Successfully merging this pull request may close these issues.

None yet

1 participant