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

Preparing release version 3.3.1 #2997

Merged
merged 1 commit into from Dec 6, 2017

Conversation

nicoddemus
Copy link
Member

@nicoddemus nicoddemus commented Dec 4, 2017

Builds:

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.7% when pulling 846bbfb on nicoddemus:release-3.3.1 into 2e8b0a8 on pytest-dev:master.

@nicoddemus
Copy link
Member Author

Created a new 3.3.1 package with the latest changes on master (specifically the pytest-capturelog blacklisting).

If this gets approved and the builds finish until bed time tonight, I can push the package later today, otherwise we will publish this tomorrow. 👍

@nicoddemus
Copy link
Member Author

Builds:

@The-Compiler
Copy link
Member

LGTM FWIW!

@RonnyPfannschmidt
Copy link
Member

is the metadata correct in the generated wheel?

@nicoddemus
Copy link
Member Author

The wheel contains this line in the METADATA file:

Requires-Python: >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*

So it seems alright. Where is the metadata for "requires python" located in the sdist?

@RonnyPfannschmidt
Copy link
Member

@nicoddemus should be in PKG-INFO at best, but usually setup.py itself

@RonnyPfannschmidt
Copy link
Member

also 👍 yay

@nicoddemus
Copy link
Member Author

should be in PKG-INFO at best, but usually setup.py itself

Ahh OK, I see it in PKG-INFO:

Requires-Python: >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*

So if all is well with you @RonnyPfannschmidt I will push the package to PyPI once the devpi builds pass. 👍

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.7% when pulling 2a111ff on nicoddemus:release-3.3.1 into 5c6758f on pytest-dev:master.

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

4 participants