Closed
Bug 968959
Opened 12 years ago
Closed 12 years ago
Clean up the repos that use the push_printurls hook
Categories
(Developer Services :: Mercurial: hg.mozilla.org, defect)
Developer Services
Mercurial: hg.mozilla.org
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: RyanVM, Assigned: RyanVM)
References
Details
Attachments
(1 file)
|
5.47 KB,
patch
|
hwine
:
review+
RyanVM
:
checked-in+
|
Details | Diff | Splinter Review |
This is horribly out of date. Of course, it would be better if we just used it everywhere, but I'll take what I can get for now.
| Assignee | ||
Comment 1•12 years ago
|
||
I reorganized things into more logical groups, added a lot of repos that are in active use, and removed repos that are not in use anymore.
Attachment #8371675 -
Flags: review?(hwine)
Comment on attachment 8371675 [details] [diff] [review]
Remove obsolete repos and add current repos to the push_printurls hook
Review of attachment 8371675 [details] [diff] [review]:
-----------------------------------------------------------------
lgtm - my opinions are blocking.
::: mozhghooks/push_printurls.py
@@ -12,5 @@
> - 'mozilla-aurora': 'releases/mozilla-aurora/',
> - 'mozilla-beta': 'releases/mozilla-beta/',
> - 'mozilla-release': 'releases/mozilla-release/',
> - 'mozilla-esr10': 'releases/mozilla-esr10/',
> - 'mozilla-esr17': 'releases/mozilla-esr17/',
esr10 & esr17 aren't set to build anymore, but this hook is about printing nice URLs on commits.
I don't see a reason to pull them, but just my opinion
@@ -24,5 @@
> - 'comm-aurora': 'releases/comm-aurora/',
> - 'comm-beta': 'releases/comm-beta/',
> - 'comm-release': 'releases/comm-release/',
> - 'comm-esr10': 'releases/comm-esr10/',
> - 'comm-esr17': 'releases/comm-esr17/',
ditto
Attachment #8371675 -
Flags: review?(hwine) → review+
| Assignee | ||
Comment 3•12 years ago
|
||
Comment on attachment 8371675 [details] [diff] [review]
Remove obsolete repos and add current repos to the push_printurls hook
Thanks for the quick review :)
https://hg.mozilla.org/hgcustom/hghooks/rev/d5e45273fd1d
Attachment #8371675 -
Flags: checked-in+
| Assignee | ||
Comment 4•12 years ago
|
||
Deployed!
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: Release Engineering → Developer Services
You need to log in
before you can comment on or make changes to this bug.
Description
•