-
Notifications
You must be signed in to change notification settings - Fork 32
Release Procedures
We assume you have a local clone of your fork of https://github.com/spacetelescope/stcal and that origin
points to your fork and upstream
points the central repo. I.e. the following (or ssh equivalents):
$ git remote -v
origin https://github.com/jdavies-st/stcal.git (fetch)
origin https://github.com/jdavies-st/stcal.git (push)
upstream https://github.com/spacetelescope/stcal.git (fetch)
upstream https://github.com/spacetelescope/stcal.git (push)
These steps should be undertaken on the main branch:
-
Update
CHANGES.rst
to have the correct release version and date. Change "unreleased" next to the intended release to the current date in YYYY-MM-DD format. -
Create a PR against
upstream/main
and merge it.
If you're making a major or minor version release, then the release branch will not yet exist. If you're releasing a patch version, then a release branch will already exist and patch may include everything on master
or need cherry-picking. Following one of the next 3 options accordingly. The following steps assume that the spacetelescope/stcal
remote is named upstream
, just as above.
-
Update pointer to
main
:$ git fetch upstream $ git checkout main $ git pull upstream main
-
Verify that the last commit in the log is as expected:
$ git log
-
Create a new release branch. The name of the release branch should share the major and minor version of your release version, but the patch version should be x. For example, when releasing
0.16.0
, name the branch0.16.x
.$ git checkout -b a.b.x upstream/main
-
Push the branch to the upstream remote.
$ git push -u upstream HEAD
Github actions CI should notice the updated branch and run the tests which you can check here. Wait until the build passes before proceeding.
If your patch release will include everything on main, the procedure is very similar to a major/minor release, but the release branch will already exist. So if you need to deliver a 0.16.1
patch to 0.16.0
, then you will do your work in branch 0.16.x
.
-
Checkout and freshen release branch (this assumes that your local branch is already tracking
upstream/a.b.x
):$ git checkout a.b.x $ git pull
-
Pull all changes on
main
into the release branch:$ git fetch upstream $ git pull upstream main
-
Push updates to the upstream remote:
$ git push upstream HEAD
Github actions CI should notice the updated branch and run the tests which you can check here. Wait until the build passes before proceeding.
In the case of a patch release, the release branch will already exist. So if you need to deliver a 0.16.1
patch to 0.16.0
, then you will do your work in branch 0.16.x
.
-
Checkout and freshen release branch (this assumes that your local branch is already tracking
upstream/a.b.x
):$ git checkout a.b.x $ git pull
-
Cherry-pick relevant commits from main that should be included in the patch release (including the new changelog commit):
$ git cherry-pick ...
-
Push updates to the upstream remote:
$ git push upstream HEAD
Github actions CI should notice the updated branch and run the tests which you can check here. Wait until the build passes before proceeding.
A bug needs to be fixed in 0.16.0. It is also present on main.
Fix the bug on main
through a PR. Merge it and get the hash of the commit. As an example, let's say the commit hash is f41120e
.
Cherry pick all commits that need to be included in the patch release.
$ git cherry-pick -x f41120e
If there are conflicts resolve them before you continue with other commits. In general start with the oldest commit so that conflicts are minimized.
Sometimes a commit is a merge commit with more than one parent. The above command will fail with an error message:
error: Commit f41120e is a merge but no -m option was given.
fatal: cherry-pick failed
This discussion is helpful:
In this case look at the parents and choose which one is the ancestor. The command then is
git cherry-pick -x -m 1 f41120e
This tells git to use the first parent.
Use git show
to verify what changes are going in the branch.
Github actions CI should notice the updated branch and run the tests which you can check here. Wait until the build passes before proceeding.
At this point, you should have the release branch checked out and ready to tag.
-
Create an annotated tag with a name that matches your intended release and a useful message. For our purposes, having the DMS Build number that this tag goes with is the current useful message:
$ git tag -a a.b.c -m "Tagged a.b.c from main"
Before proceeding, you will need to pip install twine build
.
-
Checkout the release tag, clean the directory, and make sure umask and permissions are set correctly:
$ git checkout a.b.c $ git clean -xdf $ umask 0022 $ chmod -R a+Xr .
-
Check the package setup and create the package sdist and wheel:
$ python -m build .
-
Upload the package to PyPi's test server (you need an account and be added as maintainer):
$ twine check --strict dist/* $ twine upload --repository testpypi dist/*
-
(Needs editing. Current tests are run from JWST using STCAL as a dependency). Check that it looks good on the test server. Make sure it installs without errors in a new virtual env:
$ pip install -i https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple stcal[test]
-
Run the tests on the installed package.
$ pytest --pyargs stcal
-
If the package looks good on test.pypi.org and installs OK and the tests pass, then proceed.
-
If publishing to the PyPi test server above worked well, push the new tag to the upstream remote on Github:
$ git push upstream a.b.c
-
Go to https://github.com/spacetelescope/stcal/releases and draft a new release with the existing
a.b.c
tag used above. Use a release title "Tagged a.b.c from main". When saved, this will kick off the Github action to deliver the package to Pypi officially. -
Check that it was delivered to Pypi (https://pypi.org/project/stcal/).
The stable
branch points to the latest official release of stcal
. If the current release has become the latest, then the next step is to rewrite the stable branch to point our new tag.
$ git checkout stable
$ git reset --hard a.b.c
$ git push upstream stable --force
Create and merge a PR to master
updating CHANGES.rst
, opening it up for new features and bugfixes. The diff should look something like this
$ git diff
diff --git a/CHANGES.rst b/CHANGES.rst
index e8771c7b..d7fac87e 100644
--- a/CHANGES.rst
+++ b/CHANGES.rst
@@ -1,3 +1,8 @@
+0.16.1 (unreleased)
+===================
+
+
+
0.16.0 (2020-05-04)
===================
If we're not already working on a later release on main (i.e. if we had to cherry-pick from main
onto our release branch), then we'll need to tag the HEAD of master with a development tag. This allows the setuptools-scm plugin to identify code installed from master as the latest version. I.e. this only really needs to be done if we've done a patch release on a release branch that has cherry-picked commits on it.
$ git fetch upstream
$ git checkout upstream/master
$ git tag -a a.b.d.dev -m "Tagging a.b.d.dev"
$ git push upstream a.b.d.dev