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

add MaximumLineLength rule#135

Merged
mdevils merged 1 commit intojscs-dev:masterfrom
mikesherov:maxlen
Jan 3, 2014
Merged

add MaximumLineLength rule#135
mdevils merged 1 commit intojscs-dev:masterfrom
mikesherov:maxlen

Conversation

@mikesherov
Copy link
Copy Markdown
Contributor

No description provided.

@mikesherov mikesherov restored the maxlen branch January 2, 2014 02:20
@Famlam
Copy link
Copy Markdown
Contributor

Famlam commented Jan 2, 2014

Probably this should be more like a feature request in a new issue, but can you disable this check for items that cannot be interrupted by a line break? The best (useful) example I can think of (except for var veryLongVariableNamesThatAreLongerThanTheMaximumLineLengthThatYouSpecified = true) are regexes defined as /regex/.

@mikesherov
Copy link
Copy Markdown
Contributor Author

@Famlam #20 will cover what you're talking about.

mdevils added a commit that referenced this pull request Jan 3, 2014
add MaximumLineLength rule
@mdevils mdevils merged commit ab47298 into jscs-dev:master Jan 3, 2014
@mdevils
Copy link
Copy Markdown
Member

mdevils commented Jan 3, 2014

Thanks!

@mikesherov mikesherov deleted the maxlen branch January 5, 2014 17:32
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.

3 participants