Skip to content
This repository has been archived by the owner on Nov 18, 2022. It is now read-only.

Fix use_crate_blacklist error #643

Closed
wants to merge 1 commit into from
Closed

Fix use_crate_blacklist error #643

wants to merge 1 commit into from

Conversation

arlyon
Copy link

@arlyon arlyon commented Aug 11, 2019

Fixes issue #639

bors added a commit to rust-lang/rls that referenced this pull request Aug 11, 2019
Warn against deprecated config entries

Addresses rust-lang/vscode-rust#643.

Brings back removed option from #1509 but only not to trigger the unknown option warning.
bors added a commit to rust-lang/rls that referenced this pull request Aug 11, 2019
Warn against deprecated config entries

Addresses rust-lang/vscode-rust#643.

Brings back removed option from #1509 but only not to trigger the unknown option warning.
@Xanewok
Copy link
Member

Xanewok commented Aug 11, 2019

Thanks for the PR! I don't think this is a correct fix - we should warn the user about deprecated option (which is what rust-lang/rls#1539 does) instead.

As such, I'll close this (see #639 (comment) for more details)

@Xanewok Xanewok closed this Aug 11, 2019
matklad added a commit to matklad/vscode-rust that referenced this pull request Jul 13, 2020
643: add gc request r=matklad a=matklad



Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>
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