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

Auto closing quotes in makefiles (*.mk) #89191

Closed
ZYinMD opened this issue Jan 23, 2020 · 5 comments
Closed

Auto closing quotes in makefiles (*.mk) #89191

ZYinMD opened this issue Jan 23, 2020 · 5 comments
Assignees
Labels
author-verification-requested Issues potentially verifiable by issue author feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@ZYinMD
Copy link

ZYinMD commented Jan 23, 2020

I found a tiny issue: quotes don't auto-close in makefiles (*.mk), but brackets do. It's because makefile didn't define any surroundingPairs or autoClosingPairs in its language-configuration.json, but did define brackets, and the fallback logic is here.

I'm not an expert, but I think makefile does use double and single quotes for strings.

@vscodebot
Copy link

vscodebot bot commented Jan 23, 2020

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

@alexr00 alexr00 added this to the February 2020 milestone Jan 29, 2020
@alexr00 alexr00 added the feature-request Request for new features or functionality label Jan 29, 2020
@alexr00 alexr00 modified the milestones: February 2020, Backlog Feb 4, 2020
@alexr00 alexr00 modified the milestones: Backlog, November 2020 Nov 5, 2020
@alexr00 alexr00 closed this as completed in 520a050 Nov 5, 2020
@alexr00 alexr00 added the verification-needed Verification of issue is requested label Nov 30, 2020
@rzhao271 rzhao271 added the author-verification-requested Issues potentially verifiable by issue author label Dec 2, 2020
@rzhao271
Copy link
Contributor

rzhao271 commented Dec 2, 2020

Brackets don't autocomplete for me now in makefiles, but I'm not sure if part of the original feature request was to keep that behaviour, hence author-verification-requested.

@github-actions
Copy link

github-actions bot commented Dec 2, 2020

This bug has been fixed in to the latest release of VS Code Insiders!

@ZYinMD, you can help us out by confirming things are working as expected in the latest Insiders release. If things look good, please leave a comment with the text /verified to let us know. If not, please ensure you're on version f4187f2 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.

Happy Coding!

@jrieken jrieken added the verification-found Issue verification failed label Dec 3, 2020
@jrieken jrieken reopened this Dec 3, 2020
@jrieken
Copy link
Member

jrieken commented Dec 3, 2020

I have a test.mk file, it shows Makefile at the bottom but when typing any of (,[,{ no auto closing happens

@alexr00 alexr00 closed this as completed in a63786d Dec 3, 2020
@alexr00
Copy link
Member

alexr00 commented Dec 3, 2020

Thanks for catching!

@alexr00 alexr00 removed the verification-found Issue verification failed label Dec 3, 2020
@bpasero bpasero added the verified Verification succeeded label Dec 4, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jan 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
author-verification-requested Issues potentially verifiable by issue author feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants
@bpasero @jrieken @rzhao271 @ZYinMD @alexr00 and others