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] Instruct installing Visual Studio Build Tools 2017 #3378

Merged
merged 1 commit into from
Apr 24, 2019

Conversation

micbou
Copy link
Collaborator

@micbou micbou commented Apr 23, 2019

As suggested by @bstaletic in #3376 (comment), Visual Studio Build Tools is sufficient to build YCM. Users are told to download the 2017 version since 2019 is not yet supported (require a change to the build.py script and waiting for AppVeyor to release the image).

Closes #3376.


This change is Reviewable

Visual Studio Build Tools is sufficient to build YCM. Version 2019 is
not yet supported.
@codecov
Copy link

codecov bot commented Apr 23, 2019

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3378   +/-   ##
=======================================
  Coverage   94.52%   94.52%           
=======================================
  Files          20       20           
  Lines        2063     2063           
=======================================
  Hits         1950     1950           
  Misses        113      113

Copy link
Collaborator

@bstaletic bstaletic left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 2 of 2 files at r1.
Reviewable status: 1 of 2 LGTMs obtained

Copy link
Member

@puremourning puremourning left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! 2 of 2 LGTMs obtained

@bstaletic
Copy link
Collaborator

@zzbot r+

@zzbot
Copy link
Contributor

zzbot commented Apr 24, 2019

📌 Commit 1efcfe4 has been approved by bstaletic

@zzbot
Copy link
Contributor

zzbot commented Apr 24, 2019

⌛ Testing commit 1efcfe4 with merge 2ae0044...

zzbot added a commit that referenced this pull request Apr 24, 2019
[READY] Instruct installing Visual Studio Build Tools 2017

As suggested by @bstaletic in #3376 (comment), Visual Studio Build Tools is sufficient to build YCM. Users are told to download the 2017 version since 2019 is not yet supported (require a change to the `build.py` script and [waiting for AppVeyor to release the image](appveyor/ci#2907)).

Closes #3376.

<!-- 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/3378)
<!-- Reviewable:end -->
@zzbot
Copy link
Contributor

zzbot commented Apr 24, 2019

💔 Test failed - status-travis

@zzbot
Copy link
Contributor

zzbot commented Apr 24, 2019

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

@zzbot zzbot merged commit 1efcfe4 into ycm-core:master Apr 24, 2019
@micbou micbou deleted the visual-studio-build-tools branch April 24, 2019 23:00
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.

unnecessary actions in install.py [question] [feature request]
4 participants