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

Cleanup some dependencies in tox.ini targets on Python versions #977

Merged
merged 3 commits into from
Mar 24, 2018

Conversation

rodrigc
Copy link
Contributor

@rodrigc rodrigc commented Mar 24, 2018

https://twistedmatrix.com/trac/ticket/9394

  • apidocs must run only on Python 2, because epydoc and pydoctor do not work on Python 3,
    so add a clarifying comment
  • manifest-checker, pycodestyle, pycodestylediff work on Python 2 or 3, so remove basepython
    from their targets
  • twistedchecker doesn't work on Python 3.6, and must be on Python 3.5, so change basepython
    version accordingly

 * apidocs must run only on Python 2, because epydoc and pydoctor do not work on Python 3,
   so add a clarifying comment
 * manifest-checker, pycodestyle, pycodestylediff work on Python 2 or 3, so remove basepython
   from their targets
 * twistedchecker doesn't work on Python 3.6, and must be on Python 3.5, so change basepython
   version accordingly
@rodrigc rodrigc changed the title Cleanup some dependencies on targets on Python versions Cleanup some dependencies in tox.ini targets on Python versions Mar 24, 2018
skip_install = true

# TwistedChecker needs to be 3.5
Copy link
Member

Choose a reason for hiding this comment

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

@rodrigc rodrigc merged commit 1697e42 into trunk Mar 24, 2018
@rodrigc rodrigc deleted the 9394-rodrigc-tox-fix branch March 24, 2018 23:36
@codecov
Copy link

codecov bot commented Mar 31, 2018

Codecov Report

Merging #977 into trunk will increase coverage by 0.28%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            trunk     #977      +/-   ##
==========================================
+ Coverage   91.72%      92%   +0.28%     
==========================================
  Files         843      844       +1     
  Lines      149350   155196    +5846     
  Branches    13076    14003     +927     
==========================================
+ Hits       136986   142793    +5807     
- Misses      10250    10292      +42     
+ Partials     2114     2111       -3

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