Closed mauritsvanrees closed 6 years ago
Looks good. There are a few http instead of https links, though. Can you fix those?
Reinout (Verstuurd vanaf iPhone, dus kort)
Op 15 mei 2018 om 09:47 heeft Maurits van Rees notifications@github.com het volgende geschreven:
One real code change, the rest is documentation or tests.
Fixes issue #281.
You can view, comment on, or merge this pull request online at:
https://github.com/zestsoftware/zest.releaser/pull/282
Commit Summary
Use pypi.org, especially when checking if a package is on PyPI. File Changes
M CHANGES.rst (6) M README.rst (14) M doc/source/entrypoints.rst (2) M doc/source/uploading.rst (17) M zest/releaser/release.py (2) M zest/releaser/tests/functional.py (2) Patch Links:
https://github.com/zestsoftware/zest.releaser/pull/282.patch https://github.com/zestsoftware/zest.releaser/pull/282.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.
Good catch. Done.
Travis is moving open source projects from travis-ci.org to travis-ci.com. See Travis blog. Apparently this means that currently we do not get any checks from Travis on PRs.
On the github repo there is a new Checks tab, where you can register the repo for the new travis-ci.com, see blog. But it seems that then you lose the build history. They will offer migration including history somewhere this quarter, and doing that automatically I think.
It seems strange that this all means that we have no CI currently. Maybe I am missing something.
Anyway, seems fine to merge this PR, as I don't see a chance for possible breakage here, and the tests pass locally. I will merge.
Released in zest.releaser 6.15.0.
One real code change, the rest is documentation or tests.
Fixes issue #281.