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

Fix error messages for whitespaceBetween #950

Closed
wants to merge 2 commits into from
Closed

Fix error messages for whitespaceBetween #950

wants to merge 2 commits into from

Conversation

hzoo
Copy link
Member

@hzoo hzoo commented Jan 24, 2015

For #949.

Since the rules enforce only one space.

Also removed spaces: 1 for noWhitespaceBetween since it doesn't use that as an option?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.14% when pulling 245dccb on hzoo:whitespace-message into b370466 on jscs-dev:master.

@qfox
Copy link
Member

qfox commented Jan 24, 2015

But... spaces : 1 is a good thing! 😢

@mikesherov
Copy link
Contributor

@zxqfox @hzoo removed it from _no_WhiteSpaceBetween, because the number is always 0. This LGTM. Landing.

@hzoo hzoo deleted the whitespace-message branch October 12, 2015 02:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants