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

Clean tox and fix build break (pypy3 again) #1019

Merged
merged 16 commits into from
Sep 3, 2019
Merged

Clean tox and fix build break (pypy3 again) #1019

merged 16 commits into from
Sep 3, 2019

Conversation

texasaggie97-zz
Copy link
Contributor

@texasaggie97-zz texasaggie97-zz commented Aug 29, 2019

  • This contribution adheres to CONTRIBUTING.md.
  • I've updated CHANGELOG.md if applicable.
  • I've added tests applicable for this pull request

What does this Pull Request accomplish?

List issues fixed by this Pull Request below, if any.

What testing has been done?

  • Unit
  • System

@codecov
Copy link

codecov bot commented Aug 29, 2019

Codecov Report

Merging #1019 into master will decrease coverage by 0.14%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1019      +/-   ##
==========================================
- Coverage   91.47%   91.33%   -0.15%     
==========================================
  Files          23       23              
  Lines        3601     3542      -59     
==========================================
- Hits         3294     3235      -59     
  Misses        307      307
Impacted Files Coverage Δ
generated/nifake/session.py 97.56% <0%> (-0.18%) ⬇️
generated/nimodinst/_converters.py 96.59% <0%> (-0.06%) ⬇️
generated/nifake/_converters.py 96.9% <0%> (-0.05%) ⬇️

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 822a370...b3005f0. Read the comment docs.

tools/build_release.py Show resolved Hide resolved
@marcoskirsch marcoskirsch merged commit 15958cc into master Sep 3, 2019
@marcoskirsch marcoskirsch deleted the clean_tox branch September 3, 2019 21:52
marcoskirsch pushed a commit that referenced this pull request Sep 3, 2019
* Remove numpy restriction

Bug fixed

* Upgrade pip

* Remove specific version of flake8

'\' in comments now need to be escaped

* Add basepython section

* Try using pypy3.5

* Go back to pypy3 and remove basepython block

* Don't need to run build_test for all builds - only 3.6

* Remove warning from coverage

* Undo previous change

* Try disabling faulthandler

* Remove --no-faulthandler

* pypy needs an older version of pytest

See pytest-dev/pytest#5807

* Use platform_python_implementation since implementation_name is not defined on Python2

* Try to get back to the initial coverage

* Do the same thing for build_test

* Remove redundant pip

* Add codecov file to try to control limits for passing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants