Merge lp:~cjwatson/launchpad/remove-gitnamespace-markers into lp:launchpad

[Colin Watson]

Remove the ##GITNAMESPACE## markers, now that we're on production.

Now that we're on production and have a stable URL format, there isn't much point in having the ##GITNAMESPACE## markers any more.

[William Grant (wgrant)]

Is there much point in removing them?

Also, after discussions in Austin don't we probably want to add target alternate repos?

review: Needs Information

[Jes Slow (cluelesscoder)]

Why leave a branch open from 2015? All this cruft makes it hard for community members to poke around and see what's happening. More incentive to find another community...

[Colin Watson (cjwatson)]

Yeah, I'll close this for now; I suppose these markers may yet be useful information if we decide to make some of the plausible extensions.

Unmerged revisions

17475. By Colin Watson on 2015-05-01

Remove the ##GITNAMESPACE## markers, now that we're on production.

=== modified file 'lib/lp/code/interfaces/gitnamespace.py'
--- lib/lp/code/interfaces/gitnamespace.py	2015-04-28 23:25:08 +0000
+++ lib/lp/code/interfaces/gitnamespace.py	2015-05-01 15:38:18 +0000
@@ -188,7 +188,6 @@
         return getUtility(IGitNamespaceSet).get(owner)
 
 
-# Marker for references to Git URL layouts: ##GITNAMESPACE##
 def split_git_unique_name(unique_name):
     """Return the namespace and repository names of a unique name."""
     try:

=== modified file 'lib/lp/code/interfaces/gitrepository.py'
--- lib/lp/code/interfaces/gitrepository.py	2015-04-28 16:39:15 +0000
+++ lib/lp/code/interfaces/gitrepository.py	2015-05-01 15:38:18 +0000
@@ -350,7 +350,6 @@
         More important related objects are sorted first.
         """
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     def getRepositoryIdentities():
         """A list of aliases for a repository.
 
@@ -566,7 +565,6 @@
             to the target's code sharing policy.  (optional)
         """
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     @call_with(user=REQUEST_USER)
     @operation_parameters(
         path=TextLine(title=_("Repository path"), required=True))

=== modified file 'lib/lp/code/model/gitlookup.py'
--- lib/lp/code/model/gitlookup.py	2015-03-30 14:47:22 +0000
+++ lib/lp/code/model/gitlookup.py	2015-05-01 15:38:18 +0000
@@ -74,7 +74,6 @@
 
     implements(IGitTraversable)
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     def traverse(self, owner, name, segments):
         """See `IGitTraversable`.
 
@@ -107,7 +106,6 @@
     def __init__(self, context):
         self.context = context
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     def traverse(self, owner, name, segments):
         """See `IGitTraversable`.
 
@@ -152,7 +150,6 @@
     adapts(IDistribution)
     implements(IGitTraversable)
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     def traverse(self, owner, name, segments):
         """See `IGitTraversable`.
 
@@ -204,7 +201,6 @@
     def getNamespace(self, owner):
         return getUtility(IGitNamespaceSet).get(owner)
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     def traverse(self, owner, name, segments):
         """See `IGitTraversable`.
 

=== modified file 'lib/lp/code/model/gitnamespace.py'
--- lib/lp/code/model/gitnamespace.py	2015-04-28 23:25:08 +0000
+++ lib/lp/code/model/gitnamespace.py	2015-05-01 15:38:18 +0000
@@ -249,7 +249,6 @@
             GitRepository.distribution == None,
             GitRepository.sourcepackagename == None)
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     @property
     def name(self):
         """See `IGitNamespace`."""
@@ -324,7 +323,6 @@
             GitRepository.owner == self.owner,
             GitRepository.project == self.project)
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     @property
     def name(self):
         """See `IGitNamespace`."""
@@ -411,7 +409,6 @@
             GitRepository.distribution == dsp.distribution,
             GitRepository.sourcepackagename == dsp.sourcepackagename)
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     @property
     def name(self):
         """See `IGitNamespace`."""

=== modified file 'lib/lp/code/model/gitrepository.py'
--- lib/lp/code/model/gitrepository.py	2015-04-28 16:39:15 +0000
+++ lib/lp/code/model/gitrepository.py	2015-05-01 15:38:18 +0000
@@ -209,7 +209,6 @@
         self.owner_default = False
         self.target_default = False
 
-    # Marker for references to Git URL layouts: ##GITNAMESPACE##
     @property
     def unique_name(self):
         names = {"owner": self.owner.name, "repository": self.name}