Skip to content

Commit

Permalink
Fix publish action (#264)
Browse files Browse the repository at this point in the history
The latest release didn't successfully publish; this fixes the errors from the GitHub actions.

For Python 2, encoding is not a valid keyword for open, so removed that. And for Python 3, PyPi returned a 400 status code saying Development Status :: 5 - Stable is an invalid input, so looked up the new string and added that. Also temporarily updated the workflow to run on PR, it successfully ran and published before this landed.

Co-authored-by: Scott Erickson <serickson@dropbox.com>
  • Loading branch information
sderickson and Scott Erickson committed Jan 24, 2022
1 parent 1f81db2 commit f84a8a7
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
# full Pip syntax is not supported. See also
# <http://stackoverflow.com/questions/14399534/>.
install_reqs = []
with open('requirements.txt', encoding='utf-8') as f:
with open('requirements.txt') as f: # pylint: disable=W1514
install_reqs += f.read().splitlines()

setup_requires = [
Expand All @@ -31,10 +31,10 @@
# full Pip syntax is not supported. See also
# <http://stackoverflow.com/questions/14399534/>.
test_reqs = []
with open('test/requirements.txt', encoding='utf-8') as f:
with open('test/requirements.txt') as f: # pylint: disable=W1514
test_reqs += f.read().splitlines()

with open('README.rst', encoding='utf-8') as f:
with open('README.rst') as f: # pylint: disable=W1514
README = f.read()

dist = setup(
Expand Down Expand Up @@ -67,7 +67,7 @@
maintainer='Dropbox',
url='https://github.com/dropbox/stone',
classifiers=[
'Development Status :: 5 - Stable',
'Development Status :: 5 - Production/Stable',
'Intended Audience :: Developers',
'License :: OSI Approved :: MIT License',
'Operating System :: OS Independent',
Expand Down

0 comments on commit f84a8a7

Please sign in to comment.