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] Add -include-pch flag to the list of include flags #368

Merged
merged 1 commit into from
Feb 17, 2016

Conversation

micbou
Copy link
Collaborator

@micbou micbou commented Feb 16, 2016

@Valloric
Copy link
Member

Thanks!

@homu r+

@homu
Copy link
Contributor

homu commented Feb 16, 2016

📌 Commit 187b1ec has been approved by Valloric

homu added a commit that referenced this pull request Feb 16, 2016
[READY] Add -include-pch flag to the list of include flags

Fix issue ycm-core/YouCompleteMe#1974.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/368)
<!-- Reviewable:end -->
@homu
Copy link
Contributor

homu commented Feb 16, 2016

⌛ Testing commit 187b1ec with merge 5acf21c...

@homu
Copy link
Contributor

homu commented Feb 16, 2016

💔 Test failed - status

@micbou
Copy link
Collaborator Author

micbou commented Feb 17, 2016

@homu retry


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


Comments from the review on Reviewable.io

@micbou
Copy link
Collaborator Author

micbou commented Feb 17, 2016

@homu r+


Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from the review on Reviewable.io

@homu
Copy link
Contributor

homu commented Feb 17, 2016

📌 Commit 5696336 has been approved by micbou

@homu
Copy link
Contributor

homu commented Feb 17, 2016

⚡ Test exempted - status

@homu homu merged commit 5696336 into ycm-core:master Feb 17, 2016
homu added a commit that referenced this pull request Feb 17, 2016
[READY] Add -include-pch flag to the list of include flags

Fix issue ycm-core/YouCompleteMe#1974.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/368)
<!-- Reviewable:end -->
@micbou micbou deleted the include-pch-flag branch February 20, 2016 13:26
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.

3 participants