-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 env variable to suppress completion descriptions on create #1938
Add env variable to suppress completion descriptions on create #1938
Conversation
Thanks @scop ! Instead we could have the go code itself check the new variable and not return descriptions with the list of completions. This is what we do for activehelp and probably what I should have done for descriptions from the start. This go logic would be here: Line 200 in 4dd4b25
|
I agree we should follow what active helps does , this includes support for two env vars, i.e. |
Makes sense. I suggest we take a look at #1940 before reworking this though -- if/after that's in, the env var prefix handling could be generalized and shared between active help and this feature. |
4474cf6
to
2c46d19
Compare
I suppose this is starting to take shape, however it requires #1940, so keeping as a draft until that's in. This will also cause conflicts with #1957, so either this or that will eventually need rebasing. This one's still blocked and #1957 is not, so perhaps that one could be applied first? Then again, I would not want to delay this one any more. |
2c46d19
to
5c206d1
Compare
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Similar to the existing one for active help, but generalized.
5c206d1
to
97b7001
Compare
Resolved conflicts, rebased. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your patience @scop
00019d5
to
c96f1a6
Compare
977823c
to
1107319
Compare
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Thanks for the review, @marckhouzam, feedback addressed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice!
Thanks @scop.
Closes #1937