on master help sign in
Run clippy on workspace in CI (#4614)
commiteddf9f0b7f2eacac690afad05abdae398bb17366 authorJonathan LEI <me@xjonathan.dev> dateMon 7 Nov 2022, 12:39 (4 years ago) parent140df92
3 files changed+4−3
.github/workflows/build.yml+1−1 helix-core/src/test.rs+1 xtask/src/querycheck.rs+2−2
.github/workflows/build.yml+1−1
@@ -101,7 +101,7 @@ jobs:
101101 uses: actions-rs/cargo@v1
102102 with:
103103 command: clippy
104− args: --all-targets -- -D warnings
104+ args: --workspace --all-targets -- -D warnings
105105
106106 - name: Run cargo doc
107107 uses: actions-rs/cargo@v1
helix-core/src/test.rs+1
@@ -148,6 +148,7 @@ pub fn plain(s: &str, selection: Selection) -> String {
148148}
149149
150150#[cfg(test)]
151+#[allow(clippy::module_inception)]
151152mod test {
152153 use super::*;
153154
xtask/src/querycheck.rs+2−2
@@ -17,8 +17,8 @@ pub fn query_check() -> Result<(), DynError> {
1717 let language_name = &language.language_id;
1818 let grammar_name = language.grammar.as_ref().unwrap_or(language_name);
1919 for query_file in query_files {
20− let language = get_language(&grammar_name);
21− let query_text = read_query(&language_name, query_file);
20+ let language = get_language(grammar_name);
21+ let query_text = read_query(language_name, query_file);
2222 if let Ok(lang) = language {
2323 if !query_text.is_empty() {
2424 if let Err(reason) = Query::new(lang, &query_text) {