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

Add option to disable signature help syntax highlights #4157

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

puremourning
Copy link
Member

@puremourning puremourning commented Jul 5, 2023

Fixes #4153


This change is Reviewable

Copy link
Member Author

@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:

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: 1 of 2 LGTMs obtained (waiting on @puremourning)

@codecov
Copy link

codecov bot commented Jul 5, 2023

Codecov Report

Merging #4157 (7d3e6db) into master (f93c2e9) will decrease coverage by 0.02%.
The diff coverage is 66.66%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4157      +/-   ##
==========================================
- Coverage   88.84%   88.82%   -0.02%     
==========================================
  Files          34       34              
  Lines        4393     4395       +2     
==========================================
+ Hits         3903     3904       +1     
- Misses        490      491       +1     

@puremourning puremourning added the Ship It! Manual override to merge a PR by maintainer label Aug 18, 2023
@mergify
Copy link
Contributor

mergify bot commented Aug 18, 2023

Thanks for sending a PR!

@mergify mergify bot merged commit a2e6736 into ycm-core:master Aug 18, 2023
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ship It! Manual override to merge a PR by maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add option to disable syntax highlighting in method signature
1 participant