>4000 LoC

Date: 2016-05-20 05:50 pm (UTC)
From: (Anonymous)
The real problem seems to be that this PR has >4000 LoC. If it were split up in logical units (separate PRs) to provide the infrastructure for this feature, it would not have been a problem.

This is what we do in Drupal, which is a much larger project, with more contributors, and more experience with this problem.
Identity URL: 
Account name:
If you don't have an account you can create one now.
HTML doesn't work in the subject.


If you are unable to use this captcha for any reason, please contact us by email at support@dreamwidth.org

Notice: This account is set to log the IP addresses of everyone who comments.
Links will be displayed as unclickable URLs to help prevent spam.


Matthew Garrett

About Matthew

Power management, mobile and firmware developer on Linux. Security developer at Google. Ex-biologist. @mjg59 on Twitter. Content here should not be interpreted as the opinion of my employer.

Expand Cut Tags

No cut tags