Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add postrelease support #37

Merged
merged 3 commits into from
Sep 3, 2019
Merged

Add postrelease support #37

merged 3 commits into from
Sep 3, 2019

Conversation

cjwatson
Copy link
Contributor

@cjwatson cjwatson commented Sep 1, 2017

This is somewhat related to #29.

My own motivation is that we have a temporary local fork of an old version of Twisted in Launchpad, currently versioned 13.0.0-p2, and as part of trying to convert from buildout to pip (which is a prerequisite for upgrading to a modern Twisted, for tedious reasons) I discovered that the version we have isn't PEP 440-compliant. That caused me to look into whether Twisted's current versioning code supported PEP 440 postreleases, and discovered that it doesn't; hence this PR.

@hawkowl
Copy link
Member

hawkowl commented Sep 9, 2017

There's one pyflakes error, could you squash that? The python 3.3 one is unrelated, I'll fix that on another branch.

1 similar comment
@hawkowl
Copy link
Member

hawkowl commented Sep 10, 2017

There's one pyflakes error, could you squash that? The python 3.3 one is unrelated, I'll fix that on another branch.

@codecov
Copy link

codecov bot commented Sep 11, 2017

Codecov Report

Merging #37 into master will increase coverage by 0.4%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #37     +/-   ##
=========================================
+ Coverage   96.46%   96.86%   +0.4%     
=========================================
  Files           5        5             
  Lines         650      733     +83     
  Branches       62       68      +6     
=========================================
+ Hits          627      710     +83     
  Misses         11       11             
  Partials       12       12
Impacted Files Coverage Δ
src/incremental/__init__.py 87.19% <100%> (+1.1%) ⬆️
src/incremental/tests/test_version.py 100% <100%> (ø) ⬆️
src/incremental/tests/test_update.py 100% <100%> (ø) ⬆️
src/incremental/update.py 98.23% <100%> (+0.11%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9913588...b4b9190. Read the comment docs.

@cjwatson
Copy link
Contributor Author

Ah, right, yes. I've sorted out the pyflakes error.

Copy link
Member

@glyph glyph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks pretty good to me! Given that the updated merge seems to have fixed the CI hiccup, I'm going to merge it.

@glyph glyph merged commit 23bcb51 into twisted:master Sep 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants