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

Disallow lines that span more than 160 columns. #4662

Merged
merged 1 commit into from
Jan 19, 2015

Conversation

Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Jan 19, 2015

The Rust style guide suggests 100, but we have too many violations in the
tree already. This check can be tightened over time.

The Rust style guide suggests 100, but we have too many violations in the
tree already. This check can be tightened over time.
@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/3780

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

bors-servo pushed a commit that referenced this pull request Jan 19, 2015
The Rust style guide suggests 100, but we have too many violations in the
tree already. This check can be tightened over time.
@jdm jdm added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 19, 2015
bors-servo pushed a commit that referenced this pull request Jan 19, 2015
The Rust style guide suggests 100, but we have too many violations in the
tree already. This check can be tightened over time.
@bors-servo bors-servo closed this Jan 19, 2015
@bors-servo bors-servo merged commit 394f816 into servo:master Jan 19, 2015
@Ms2ger Ms2ger deleted the overlong branch January 19, 2015 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants