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

[READY] Exclude psutil 5.0.1 from test requirements #695

Merged
merged 1 commit into from
Jan 22, 2017

Conversation

micbou
Copy link
Collaborator

@micbou micbou commented Jan 22, 2017

Last version of psutil (5.0.1) is broken on Python 3.6 and Windows 64-bit. See issue giampaolo/psutil#951. We exclude it from our test requirements. This fixes the builds with arch=64 and python=36 on AppVeyor.


This change is Reviewable

psutil 5.0.1 is broken on Python 3.6 and Windows 64-bit.
@codecov-io
Copy link

Current coverage is 92.76% (diff: 100%)

Merging #695 into master will not change coverage

@@             master       #695   diff @@
==========================================
  Files            79         79          
  Lines          5269       5269          
  Methods         297        297          
  Messages          0          0          
  Branches        139        139          
==========================================
  Hits           4888       4888          
  Misses          326        326          
  Partials         55         55          

Powered by Codecov. Last update 835cc39...5fecb72

@Valloric
Copy link
Member

Thanks!

LGTM

@vheon
Copy link
Contributor

vheon commented Jan 22, 2017

:lgtm: @homu r=valloric


Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@homu
Copy link
Contributor

homu commented Jan 22, 2017

📌 Commit 5fecb72 has been approved by valloric

@homu homu merged commit 5fecb72 into ycm-core:master Jan 22, 2017
@homu
Copy link
Contributor

homu commented Jan 22, 2017

⚡ Test exempted - status

homu added a commit that referenced this pull request Jan 22, 2017
[READY] Exclude psutil 5.0.1 from test requirements

Last version of psutil (5.0.1) is broken on Python 3.6 and Windows 64-bit. See issue giampaolo/psutil#951. We exclude it from our test requirements. This fixes the builds with `arch=64` and `python=36` on AppVeyor.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/695)
<!-- Reviewable:end -->
@micbou micbou deleted the windows-psutil branch February 26, 2017 00:06
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.

5 participants