Closed Bug 1127316 Opened 9 years ago Closed 9 years ago

The try commit hook is missing commits with Try syntax in them

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: RyanVM, Assigned: gps)

References

Details

Attachments

(1 file)

See the link below. Two csets in the push, both with Try syntax, went through without an issue. Presumably a regression from bug 1087328?

https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=66f0f0b2bfd2
Flags: needinfo?(gps)
When searching for Try syntax in commit messages, we should look at the
entire commit message, not just the first line.

The changed test fails before the corresponding hook change.
Attachment #8556710 - Flags: review?(emorley)
Assignee: nobody → gps
Status: NEW → ASSIGNED
Attachment #8556710 - Flags: review?(emorley) → review+
Deployed to production.
Flags: needinfo?(gps)
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: