Skip to content

Conversation

@emlaver
Copy link
Contributor

@emlaver emlaver commented Feb 10, 2021

PR summary

Upload to pypi fails during release because of missing comma:

2021-02-10 14:30:52.934 |  HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/
2021-02-10 14:30:52.935 |  Invalid value for classifiers. Error: Classifier 'Programming Language :: Python :: 3.9Development Status :: 4 - Beta' is not a valid classifier.

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the
    Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • New tests
  • Build/CI related changes
  • Documentation content changes
  • Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@emlaver emlaver requested a review from a team as a code owner February 10, 2021 20:37
@emlaver emlaver self-assigned this Feb 10, 2021
@codecov
Copy link

codecov bot commented Feb 10, 2021

Codecov Report

Merging #76 (11c86d8) into master (ad34ab4) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #76   +/-   ##
=======================================
  Coverage   92.29%   92.29%           
=======================================
  Files           8        8           
  Lines        7085     7085           
=======================================
  Hits         6539     6539           
  Misses        546      546           

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 ad34ab4...11c86d8. Read the comment docs.

@bessbd bessbd mentioned this pull request Feb 11, 2021
13 tasks
@emlaver emlaver merged commit 9c874e4 into master Feb 11, 2021
@emlaver emlaver deleted the fix-setup-missing-comma branch February 11, 2021 14:20
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.

4 participants