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

Add the configuration options used by the plugin #89

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

marmistrz
Copy link

@marmistrz marmistrz commented Jun 17, 2018

Before that change, VS Code would complain about unknown configuration
options if those options were added to settings.json.

This is properly recognized by VS Code.
The CI failure seems unrelated.

Before that change, VS Code would complain about unknown configuration
options if those options were added to `settings.json`.
@dragos
Copy link
Owner

dragos commented Jun 18, 2018

This was supposedly done already in #71 but I have no idea what happened to that commit. I can't find it in git history, even though the PR was merged. :-/

@marmistrz
Copy link
Author

For some reason today the ensime language server refused to start when using this package.json with the current release, so remerging your change may be a better idea.

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.

2 participants