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

add option to run clippy with --all-targets --all-features #1109

Closed
matthiaskrgr opened this issue Oct 30, 2018 · 6 comments
Closed

add option to run clippy with --all-targets --all-features #1109

matthiaskrgr opened this issue Oct 30, 2018 · 6 comments

Comments

@matthiaskrgr
Copy link
Member

Currently when I want to check tests with clippy, I still have to fall back to the command line and manually run clippy.

It would be useful if rls had a configuration option that runs clippy on all targets with all features enabled instead. :)

@alexheretic
Copy link
Member

alexheretic commented Oct 31, 2018

These are all already configuration options, have you tried (however your client does config):

clippy_preference = "on"
all_features = true

Note: all_targets is default on, but also configurable. Individual features may also be selected see rls#configuration.

@norru
Copy link

norru commented Nov 5, 2018

@norru
Copy link

norru commented Nov 5, 2018

@mickaelistria

@norru
Copy link

norru commented Nov 5, 2018

#1047

@norru
Copy link

norru commented Dec 1, 2018

@Xanewok
Copy link
Member

Xanewok commented Mar 3, 2019

Closing this as already done, please reopen if the issue still persists!

@Xanewok Xanewok closed this as completed Mar 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants