Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Move away from deprecated language configuration settings #406

Closed
wants to merge 1 commit into from

Conversation

aeschli
Copy link

@aeschli aeschli commented Jul 25, 2016

In the July release of VSCode we added new properties to the language configuration files so that the deprecated __characterPairSupport and __electricCharacterSupport APIs in vscode.d.ts are no longer necessary.
See microsoft/vscode#9281.

I've already added the comment, brackets and bracket pairs to the go language definition in the built-in go extension. See microsoft/vscode@8020da6

You can keep your current state a while if you want to be backward compatible to pre-July versions.

@msftclas
Copy link

Hi @aeschli, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!


It looks like you're a Microsoft contributor (Martin Aeschlimann). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@@ -86,35 +86,7 @@ export function activate(ctx: vscode.ExtensionContext): void {
// ^.*\{[^}'']*$
increaseIndentPattern: /^.*\{[^}'']*$/
},
wordPattern: /(-?\d*\.\d\w*)|([^\`\~\!\@\#\%\^\&\*\(\)\-\=\+\[\{\]\}\\\|\;\:\'\"\,\.\<\>\/\?\s]+)/g,
Copy link
Contributor

Choose a reason for hiding this comment

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

@aeschli I get that we are deleting comments, brackets, __electricCharachterSupport and __characterPairSupport because we have the language_configuration.json in the built-in extension "vscode.go"

But why are we deleting the wordPattern here?

@aeschli
Copy link
Author

aeschli commented Oct 31, 2016

Oh, that looks like a bug. wordPattern has to stay.

@ramya-rao-a
Copy link
Contributor

Closing this in favor of #587

@ramya-rao-a ramya-rao-a deleted the aeschli-patch-1 branch November 30, 2016 20:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants