Skip to content

Adds the "Requires-Python" metadata support. Fixes #378 - #379

Merged
alexeagle merged 4 commits into
bazel-contrib:masterfrom
AutomatedTester:python_requires
Nov 13, 2020
Merged

Adds the "Requires-Python" metadata support. Fixes #378#379
alexeagle merged 4 commits into
bazel-contrib:masterfrom
AutomatedTester:python_requires

Conversation

@AutomatedTester

Copy link
Copy Markdown
Contributor

This adds support for allowing people to specify which versions
of python a wheel should run on. This was added in PEP 440

PR Checklist

Please check if your PR fulfills the following requirements:

  • Does not include precompiled binaries, eg. .par files. See CONTRIBUTING.md for info
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

This adds support for PEP 440 to wheels

  • Bugfix
  • Feature (please, look at the "Scope of the project" section in the README.md file)
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: #378

What is the new behavior?

Allows people to specify which versions of python a wheel can support.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

This adds support for allowing people to specify which versions
of python a wheel should run on. This was added in PEP 440
@AutomatedTester

Copy link
Copy Markdown
Contributor Author

Will fix failing tests in the morning

@AutomatedTester

Copy link
Copy Markdown
Contributor Author

tests are green now @andyscott @thundergolfer @alexeagle

@alexeagle
alexeagle merged commit aa27a3f into bazel-contrib:master Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants