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
hzoo wants to merge 2 commits into
jscs-dev:masterfrom
hzoo:whitespace-message
Closed

Fix error messages for whitespaceBetween#950
hzoo wants to merge 2 commits into
jscs-dev:masterfrom
hzoo:whitespace-message

Conversation

@hzoo
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Member

qfox commented Jan 24, 2015

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

@mikesherov
Copy link
Copy Markdown
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