eecs485staff / agio-cli

A command line interface to autograder.io
MIT License
1 stars 0 forks source link

Publish to PyPI #24

Closed awdeorio closed 3 years ago

awdeorio commented 3 years ago

Publish to PyPI.

Closes #2

Validation

codecov-commenter commented 3 years ago

Codecov Report

Merging #24 (c0aea81) into develop (42a75cb) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop      #24   +/-   ##
========================================
  Coverage    71.06%   71.06%           
========================================
  Files            4        4           
  Lines          432      432           
========================================
  Hits           307      307           
  Misses         125      125           

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 42a75cb...c0aea81. Read the comment docs.

japplefield commented 3 years ago
jmapple@Justins-MacBook-Pro agio-test % python3 -m venv env
jmapple@Justins-MacBook-Pro agio-test % source env/bin/activate
(env) jmapple@Justins-MacBook-Pro agio-test % pip install agiocli
Collecting agiocli
  Using cached agiocli-0.1.0-py3-none-any.whl (12 kB)
Collecting requests
  Using cached requests-2.26.0-py2.py3-none-any.whl (62 kB)
Collecting click
  Using cached click-8.0.1-py3-none-any.whl (97 kB)
Collecting pick
  Using cached pick-1.0.0-py2.py3-none-any.whl (5.5 kB)
Collecting urllib3<1.27,>=1.21.1
  Using cached urllib3-1.26.6-py2.py3-none-any.whl (138 kB)
Collecting idna<4,>=2.5
  Using cached idna-3.2-py3-none-any.whl (59 kB)
Collecting charset-normalizer~=2.0.0
  Using cached charset_normalizer-2.0.3-py3-none-any.whl (35 kB)
Collecting certifi>=2017.4.17
  Using cached certifi-2021.5.30-py2.py3-none-any.whl (145 kB)
Installing collected packages: urllib3, idna, charset-normalizer, certifi, requests, pick, click, agiocli
Successfully installed agiocli-0.1.0 certifi-2021.5.30 charset-normalizer-2.0.3 click-8.0.1 idna-3.2 pick-1.0.0 requests-2.26.0 urllib3-1.26.6
(env) jmapple@Justins-MacBook-Pro agio-test % agio -h
Traceback (most recent call last):
  File "/Users/jmapple/Developer/agio-test/env/bin/agio", line 5, in <module>
    from agiocli.__main__ import main
  File "/Users/jmapple/Developer/agio-test/env/lib/python3.9/site-packages/agiocli/__init__.py", line 4, in <module>
    from .utils import *
  File "/Users/jmapple/Developer/agio-test/env/lib/python3.9/site-packages/agiocli/utils.py", line 11, in <module>
    import dateutil.parser
ModuleNotFoundError: No module named 'dateutil'

Should setup.py be modified to require python-dateutil?

awdeorio commented 3 years ago

Good catch! Fixed 00ea4a499d0e01f272735808733aac74c3c14b1c

japplefield commented 3 years ago

Fixed tiny typo in README, does it automatically get updated on PyPI when the PR is merged? Otherwise LGTM.

awdeorio commented 3 years ago

Does it automatically get updated on PyPI when the PR is merged? Otherwise LGTM.

Nope, that's a manual process