Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upAdded rule: AvoidLongLines #1329
Conversation
|
Thanks but running this rule by default will break a lot of people. |
|
@bergmeister I have made the changes you requested :) |
|
The 2 tests need fixing: The second test is for the documentation to have a matching rule document for every rule. Somehow it does not find the [-] Every rule must have an entry in the rule documentation README.md file 26ms |
|
I've just pushed a fix for the 2 tests and pulled from master. This should make everything green now. Will add some docs and then we should be ready to go. |
thomasrayner commentedSep 3, 2019
•
edited by bergmeister
PR Summary
Addresses #1243. This rule issues a warning for lines that are longer than 120 characters. This default value can be override and the rule is not enabled by default.
PR Checklist
.cs,.ps1and.psm1files have the correct copyright headerWIP:to the beginning of the title and remove the prefix when the PR is ready.