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

Improve the settings interface and update extension dependencies #45

Merged
merged 6 commits into from
Jan 4, 2022

Conversation

awmottaz
Copy link
Contributor

@awmottaz awmottaz commented Nov 3, 2021

Updates the contributes.configuration

Screen Shot 2021-11-02 at 19 33 22

Also uninstalls the deprecated vscode package in favor of @types/vscode. The postinstall step is no longer needed. There were no actual tests of this extension, so I deleted the outdated test example files.

The implementation of getConfiguration was updated to use the document scope, so language overrides are handled by VSCode instead of the old custom implementation.

Tony Mottaz added 6 commits November 2, 2021 18:15
* The "type" field is ignored by VSCode on the root configuration object
* The title should match the exact name of the extension, omitting words
  like "configuration"
The "vscode" package is deprecated in favor of "@types/vscode" and
"vscode-test". Since no actual tests were written for this package, the
tests are deleted and the "vscode-test" dependency was omitted.

The "postinstall" script is no longer required.
@jeremychone
Copy link
Contributor

First, thank you for your contribution. Give me a few days to look at this pull request.

@jeremychone
Copy link
Contributor

Sorry for the delay, still need more time, but did not forget about this.

@jeremychone
Copy link
Contributor

Sorry again, I am going to dedicate some time this week to merge this one. Again, thanks for this great work.

@jeremychone jeremychone merged commit 06abc60 into BriteSnow:master Jan 4, 2022
@jeremychone
Copy link
Contributor

published in v0.3.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unknown editor configuration setting "togglequotes.chars"
2 participants