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

Publishing package to pypi fails. #631

Closed
OrangeTux opened this issue May 8, 2024 · 0 comments · Fixed by #636
Closed

Publishing package to pypi fails. #631

OrangeTux opened this issue May 8, 2024 · 0 comments · Fixed by #636
Assignees
Labels
bug Something isn't working

Comments

@OrangeTux
Copy link
Contributor

OrangeTux commented May 8, 2024

The GA workflow to publish the package to pypi fails.

The issue is that the README.rst is not valid.
One can check it's validity using:

$ python3 -m pip install build, twine
$ python3 -m build --sdist
$ twine check dist/*
Checking dist/ocpp-1.0.0.tar.gz: FAILED
ERROR    `long_description` has syntax errors in markup and would not be rendered on PyPI.
         line 62: Warning: Title underline too short.

         Charging Station Management System (CSMS) / Central System
         ~~~~~~~~~~~~~~

For more information, read
We should fix the issue and modify our GA workflow to prevent this issue from happening again.

For more information, read:

After the fixes, I suggest we create a v1.01 release

@OrangeTux OrangeTux added the bug Something isn't working label May 8, 2024
OrangeTux added a commit that referenced this issue May 21, 2024
This commit fixes formatting README.rst so that PYPI accepts it.

Fixes: #631
ajmirsky pushed a commit to ajmirsky/ocpp that referenced this issue Nov 26, 2024
This commit fixes formatting README.rst so that PYPI accepts it.

Fixes: mobilityhouse#631
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging a pull request may close this issue.

2 participants