Closed Bug 968959 Opened 8 years ago Closed 8 years ago

Clean up the repos that use the push_printurls hook

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: RyanVM, Assigned: RyanVM)

References

Details

Attachments

(1 file)

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.
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+
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+
Depends on: 969014
Deployed!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Product: Release Engineering → Developer Services
You need to log in before you can comment on or make changes to this bug.