968959 - Clean up the repos that use the push_printurls hook

Status: RESOLVED FIXED

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

Description

[Ryan VanderMeulen [:RyanVM]]

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.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Attachment: Remove obsolete repos and add current repos to the push_printurls hook

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.

Comment 2

[Hal Wine [:hwine] (use NI)]

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

Comment 3

[Ryan VanderMeulen [:RyanVM]]

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

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Deployed!