Closed inclement closed 5 years ago
.travis.yml
does currently exclude some pythonforandroid/pythonpackage.py
tests (intentionally, due to long runtime). tox
runs them.
Therefore I propose that running tox
and checking that no tests fail should be added to the release checklist! It might also be a good idea to add a remark / document that this runs more test than travis for time constraint reasons, just so people understand why this is included into the checklist
Good point @JonasT, I've added it to the list. I also added that the testapps should be run under both armeabi-v7a and arm64-v8a. We'll want to do more general work to make sure arm64 is tested properly, but it should already be good to have it in the checklist.
Closing in favour of PR #1838
I'm thinking to create a release checklist and instructions that we can reuse, in preparation for acting on #1833. Once this is complete, I'll move it to proper documentation somewhere.
The following is what I currently would do to make a release. With a git flow workflow we'd want to do the same thing, except with the nice feature that the release gets its own branch to run tests and checks on - I've never been very good about that.
tox
: this performs some long-running tests that are skipped on Travis.armeabi-v7a
andarm64-v8a
:python3 setup_testapp_python3_sqlite_openssl.py apk
python3 setup_testapp_python2.py apk
python3 setup.py sdist
python3 -m twine upload
The release should also involve an announcement, which I intend to get on top of but don't consider part of this core process.
Questions:
I'm intending to test this procedure as currently written for making a release at the same time as setting up the develop branch, in order to check it works.