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] Update ycmd #3010

Merged
merged 2 commits into from
May 2, 2018
Merged

[READY] Update ycmd #3010

merged 2 commits into from
May 2, 2018

Conversation

micbou
Copy link
Collaborator

@micbou micbou commented May 2, 2018

Include the following changes:

Fixes #3001.
Fixes #3007.


This change is Reviewable

micbou added 2 commits May 2, 2018 20:35
Include the following changes:
 - cache include paths;
 - increase Python 2 requirement to 2.7.1;
 - force MSVC to treat source files as UTF-8 encoded;
 - bundle and compile the regex module;
 - use sysconfig instead of distutils.sysconfig in build script.
@bstaletic
Copy link
Collaborator

bstaletic commented May 2, 2018

Nice one. :lgtm:

@puremourning
Copy link
Member

:lgtm: good stuff.


Review status: 0 of 3 files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@codecov-io
Copy link

codecov-io commented May 2, 2018

Codecov Report

Merging #3010 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3010   +/-   ##
=======================================
  Coverage   92.84%   92.84%           
=======================================
  Files          21       21           
  Lines        2068     2068           
=======================================
  Hits         1920     1920           
  Misses        148      148

@bstaletic
Copy link
Collaborator

@zzbot r+


Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

@zzbot
Copy link
Contributor

zzbot commented May 2, 2018

📌 Commit f76c889 has been approved by bstaletic

@zzbot
Copy link
Contributor

zzbot commented May 2, 2018

⌛ Testing commit f76c889 with merge 36f716d...

zzbot added a commit that referenced this pull request May 2, 2018
[READY] Update ycmd

Include the following changes:

 - PR ycm-core/ycmd#919: cache include paths;
 - PR ycm-core/ycmd#1013: increase Python 2 requirement to 2.7.1;
 - PR ycm-core/ycmd#1015: force MSVC to treat source files as UTF-8 encoded;
 - PR ycm-core/ycmd#1017: bundle and compile the regex module;
 - PR ycm-core/ycmd#1020: use `sysconfig` instead of `distutils.sysconfig` in build script.

Fixes #3001.
Fixes #3007.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/youcompleteme/3010)
<!-- Reviewable:end -->
@zzbot
Copy link
Contributor

zzbot commented May 2, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: bstaletic
Pushing 36f716d to master...

@zzbot zzbot merged commit f76c889 into ycm-core:master May 2, 2018
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