Invoke deploy should require a tag
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 55/100
Research direction
Start at the implementation of the invoke deploy command and trace how its current checkout and deployment target are selected. Update the command so a tag is required, that tag is checked out for deployment, and the repository returns to HEAD afterward; verify that untagged deployment is rejected.
Written by the indexing model from the issue text.
Description
Right now invoke deploy deploys whatever current version is checked out. This means that unless one is careful it's easy to deploy the master branch by mistake instead of the tagged version. Moreover, it should be disallowed to deploy a version that is not properly tagged.
invoke deploy should be rewritten to require a tag argument. The code should then checkout that tag, deploy it, and revert to HEAD.
- Dominant language
- Python
- Stars
- 6
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from sdss/python_template
-
create allspec repo? Open
sdss/python_template#24 · 7 comments · 1 assignee ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 38/100
sdss/python_template#23 ·
All issues in sdss/python_template
Similar issues
-
Add: hunch Open
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
AbdelStark/awesome-typesafe#104 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
DiamondLightSource/dodal#2211 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
openml/openml-python#1749 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
sipyourdrink-ltd/bernstein#6191 ·