#4238 hooks: Only block new branches
Merged 4 months ago by pingou. Opened 4 months ago by lsedlar.
lsedlar/pagure no-block-tags  into  master

@@ -70,6 +70,10 @@ 

              return

  

          for refname in changes:

+             if refname.startswith("refs/tags/"):

+                 # Allow creating new tags

+                 continue

+ 

              (oldrev, newrev) = changes[refname]

  

              if set(oldrev) == set(["0"]):

Tags should be allowed, but any other new reference will be blocked.

Fixes: #4237

pretty please pagure-ci rebuild

4 months ago

The tests are failing locally for me as well, but it seems unrelated to this change. Only the style test fails because it wants to reformat /pagure/pagure/api/project.py.

pretty please pagure-ci rebuild

4 months ago

Ok, I'm going to merge as is and see if I can reproduce the issue locally.

I'm also working on changing how we run the test suite so it's easier to reproduce issues locally

Pull-Request has been merged by pingou

4 months ago