Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Adding jscs rules #9685

Closed
wants to merge 4 commits into from
Closed

Adding jscs rules #9685

wants to merge 4 commits into from
+149 −146

Conversation

hzoo
Copy link
Contributor

@hzoo hzoo commented Oct 20, 2014

alphabetize rules
add rule: disallowTrailingComma: 17 changes
add rule: validateParameterSeparator: 26 changes
add rule: disallowSpacesInsideParentheses: 168 changes

cc: @caitp

hzoo added 4 commits October 19, 2014 18:22
@hzoo
style(*): alphabetize jscs rules

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
0fc987b
@hzoo
style(*): add disallowTrailingComma rule for objects and arrays
442d251
@hzoo
style(*): add validateParameterSeparator rule to jscs
070e9fa
@hzoo
style(*): add disallowSpacesInsideParentheses rule to jscs
2091003
@petebacondarwin
Copy link
Contributor

LGTM - merging

petebacondarwin pushed a commit that referenced this pull request Oct 20, 2014
@hzoo @petebacondarwin
style(*): add validateParameterSeparator rule to jscs
31ec9f1 
Closes #9685
petebacondarwin pushed a commit that referenced this pull request Oct 20, 2014
@hzoo @petebacondarwin
style(*): add disallowSpacesInsideParentheses rule to jscs
483ce91 
Closes #9685
@hzoo hzoo deleted the add-jscs-rules branch October 25, 2014 03:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@hzoo @petebacondarwin @mary-poppins