Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Count line length differently #365

Merged
merged 2 commits into from Mar 7, 2017
Merged

Count line length differently #365

merged 2 commits into from Mar 7, 2017

Conversation

martinthomson
Copy link
Member

I need to check this on my mac.

@martinthomson martinthomson added the editorial An issue that does not affect the design of the protocol; does not require consensus. label Mar 7, 2017
@martinthomson martinthomson merged commit 21a0b36 into master Mar 7, 2017
@martinthomson martinthomson deleted the linecount2 branch March 7, 2017 03:25
@martinthomson
Copy link
Member Author

@janaiyengar, this should work for you. I have also changed this so that it reports exactly which line is too long, including a line number. That should make it easier to find problems.

@MikeBishop
Copy link
Contributor

I think this reverts the change I made to exempt tables from the length check, though. Kramdown requires the entire table row to be on a single line, even if it winds up wrapping the table cell contents itself later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants