Skip to content
This repository has been archived by the owner on Aug 22, 2024. It is now read-only.

Provide a way to run MIRAI as part of the linter #114

Closed
hermanventer opened this issue Apr 3, 2019 · 1 comment
Closed

Provide a way to run MIRAI as part of the linter #114

hermanventer opened this issue Apr 3, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@hermanventer
Copy link
Contributor

Issue

Running MIRAI over an entire crate every time it the crate is rebuilt is somewhat painful since every function body is re-analyzed at least once.

An alternative way to is to run MIRAI as part of the linter (see https://doc.rust-lang.org/unstable-book/language-features/plugin.html#lint-plugins and https://github.com/rust-lang/rust-clippy/blob/61aa5c957c219abe1fb2d1b5e51db8b365b4f565/clippy_lints/src/missing_const_for_fn.rs#L105).

It seems that the linter will only call the MIRAI plugin when a function has changed, so the it should be sufficient to invoke the checker in the same way as will be done by #113.

@hermanventer hermanventer added the enhancement New feature or request label Apr 3, 2019
@hermanventer
Copy link
Contributor Author

Not for v1.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant