Open Bug 1135788 Opened 6 years ago Updated 4 years ago

Commit hook for missing idl uuid revs blocks whitespace-only changes

Categories

(Developer Services :: Mercurial: hg.mozilla.org, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: Fallen, Assigned: xidorn)

References

Details

Attachments

(1 file, 1 obsolete file)

The commit hook from bug 1128518 also rejects pushes where all that changed in an IDL interface is whitespaces. I know I could add IGNORE IDL for this, but it would be nice if it could be ignored in the hook.
Assignee: nobody → quanxunzhen
Status: NEW → ASSIGNED
Attachment #8710318 - Flags: review?(gps)
This patch should make it stop prompting for most harmless whitespace changes. Some code style fix could still get stuck, though.
Attachment #8710318 - Attachment is obsolete: true
Attachment #8710318 - Flags: review?(gps)
Attachment #8710324 - Flags: review?(gps) → review+
Please mark resolved once it gets deployed.
QA Contact: hwine → klibby
You need to log in before you can comment on or make changes to this bug.