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

Bugfix dependencies #66

Merged
merged 7 commits into from
Feb 15, 2018

Conversation

BenjamenMeyer
Copy link
Member

@BenjamenMeyer BenjamenMeyer commented Feb 13, 2018

  • Bug Fix: request-mock support accidentally imported mock
  • Bug Fix: Corrected coverage configuration to ignore tests (again)
  • Bug Fix: Corrected the configuration to specify optional dependencies
  • Bug Fix: pep8 project got renamed to pycodestyle
  • Bug Fix: updated changes per pycodestyle
  • Bug Fix: corrected testing dependencies, only pep8 config needs the pycodestyle dependency not the general tests

Closes #65 #67

- Bug Fix: request-mock support accidentally imported `mock`
- Bug Fix: Corrected coverage configuration to ignore tests (again)
- Bug Fix: Corrected the configuration to specify optional dependencies
- Bug Fix: pep8 project got renamed to pycodestyle
- Bug Fix: updated changes per pycodestyle
- Bug Fix: corrected testing dependencies, only pep8 config needs
  the pycodestyle dependency not the general tests
@BenjamenMeyer BenjamenMeyer added this to the 0.13 milestone Feb 13, 2018
@BenjamenMeyer BenjamenMeyer self-assigned this Feb 13, 2018
This was referenced Feb 13, 2018
- Bug Fix: Travice-CI now requires `sudo` requirements to be
  explicitly noted.
setup.py Outdated
@@ -12,6 +17,7 @@
author='Benjamen R. Meyer',
author_email='bm_witness@yahoo.com',
install_requires=REQUIRES,
extra_requests=EXTRA_REQUIRES,
Copy link

@pglass pglass Feb 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra_requests -> extras_require

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wow...didn't catch I mistyped requires as requests - thanks for catching that one, and on the correction on where the s goes.

@BenjamenMeyer
Copy link
Member Author

@pglass updated per #67 as well.

Copy link

@pglass pglass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There could be a way to simplify the new tox config, but idk - tox.ini has always been slightly confusing to me.

@BenjamenMeyer BenjamenMeyer merged commit 0677c39 into TestInABox:master Feb 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants