GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||
GeezerVison is showing github.com — doc: change line threshold and reword subsystem test in large PR policy by Renegade334 · Pull Request #65685 · nodejs/node Conversation
|
|
Review requested:
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
For real-world context, this is a list of all of the commits in the last two years that would have triggered the line threshold after this change, but would not have done so before: Details9cf2ce7 deps: update perfetto to 57.2 Most of these are automated dependency updates that would not qualify anyway. However, the three human PRs in this list are prime examples of what should really be considered a large PR: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Signed-off-by: Renegade334 <[email protected]>
01ce2c5 to
d6a7776
Compare
|
Would anyone else from @nodejs/tsc like to weigh in on this? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.