Closed Bug 820234 Opened 12 years ago Closed 4 years ago

Warn about patches with fewer than 8 lines of context

Categories

(bugzilla.mozilla.org :: Extensions, enhancement)

Product:
bugzilla.mozilla.org
Component:
Extensions
Version:
Production
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: justin.lebar+bug, Unassigned)

Details

Now that more people are developing for Gecko with git, I'm seeing a lot of patches with 4 lines of context.

Unfortunately there's no way to configure git to export patches with 8 lines of context by default.  You have to use a custom export command, or remember to pass -U8.

There's a lot of time wasted when someone posts a patch with too little context; the reviewer wastes time, and the patch writer wastes time.

Can we warn users when they try to attach a patch with too little context?
Type: defect → enhancement
Component: Extensions: BMO → Extensions

We no longer use Bugzilla for code review.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.