Skip to content
This repository has been archived by the owner on Jun 12, 2023. It is now read-only.

tag.py broken for releases .0 releases after an rc #24

Open
TomAugspurger opened this issue Jan 29, 2020 · 6 comments
Open

tag.py broken for releases .0 releases after an rc #24

TomAugspurger opened this issue Jan 29, 2020 · 6 comments

Comments

@TomAugspurger
Copy link
Collaborator

We would tag of master rather than 1.0.x for example.

@TomAugspurger
Copy link
Collaborator Author

Hack for this release

diff --git a/scripts/tag.py b/scripts/tag.py
index 4820436..3dc4611 100755
--- a/scripts/tag.py
+++ b/scripts/tag.py
@@ -28,6 +28,7 @@ def checkout(tag):
     else:
         base = '.'.join([tag[1:].rsplit('.', 1)[0], 'x'])
 
+    base = '1.0.x'
     subprocess.check_call(['git', 'checkout', base])
     subprocess.check_call(['git', 'pull', '--ff-only', 'upstream', base])
 

@TomAugspurger
Copy link
Collaborator Author

Same for push-tag.py

diff --git a/scripts/push-tag.py b/scripts/push-tag.py
index 89c75b0..bbcdc32 100755
--- a/scripts/push-tag.py
+++ b/scripts/push-tag.py
@@ -31,6 +31,7 @@ def get_branch(tag):
         base = 'master'
     else:
         base = '.'.join([tag[1:].rsplit('.', 1)[0], 'x'])
+    base = '1.0.x'
 
     return base
 
diff --git a/scripts/tag.py b/scripts/tag.py
index 4820436..3dc4611 100755
--- a/scripts/tag.py
+++ b/scripts/tag.py
@@ -28,6 +28,7 @@ def checkout(tag):
     else:
         base = '.'.join([tag[1:].rsplit('.', 1)[0], 'x'])
 
+    base = '1.0.x'
     subprocess.check_call(['git', 'checkout', base])
     subprocess.check_call(['git', 'pull', '--ff-only', 'upstream', base])
 

@jorisvandenbossche
Copy link
Member

That reminded me, we also should "start" the 1.1 dev cycle on master

@TomAugspurger
Copy link
Collaborator Author

TomAugspurger commented Jan 30, 2020

Mmm I thought we didn't need to. I thought master already had changes for 1.1.

Actually, shouldn't we have already tagged 1.1.dev0, immediately after 1.0.0rc0?

@jorisvandenbossche
Copy link
Member

Actually, shouldn't we have already tagged 1.1.dev0, immediately after 1.0.0rc0?

Yes, that's what I meant

@TomAugspurger
Copy link
Collaborator Author

The 1.1.0.dev0 tag has been pushed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants