CI clippy didn't emit warning about too large types that were found when running clippy locally
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia con l’esecuzione CI collegata nell’issue e confronta la relativa invocazione di clippy e la toolchain Rust 1.77.2 con il comando locale che ha prodotto gli avvisi. Ispeziona rust/operator-binary/src/controller.rs alle righe 638, 792 e 1323, concentrandoti sugli avvisi result_large_err segnalati. Il lavoro è completato quando la differenza è spiegata e CI segnala gli stessi avvisi, se appropriato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
When reviewing #614 I ran clippy locally and got the warnings pasted below, but CI running clippy did not alert on these https://github.com/stackabletech/nifi-operator/actions/runs/9003755958
Probably worth investigating why this differs at some point.
My local rust:
❯ cargo --version
cargo 1.77.2 (e52e36006 2024-03-26)
Version stated in ci: RUST_TOOLCHAIN_VERSION: "1.77.2"
Warnings:
warning: the `Err`-variant returned from this function is very large
--> rust/operator-binary/src/controller.rs:638:6
|
186 | ProductConfigLoadFailed { source: config::Error },
| ------------------------------------------------- the largest variant contains at least 128 bytes
...
210 | BoostrapConfig { source: crate::config::Error },
| ----------------------------------------------- the variant `BoostrapConfig` contains at least 128 bytes
...
213 | / BuildProductConfig {
214 | | source: crate::config::Error,
215 | | rolegroup: RoleGroupRef<NifiCluster>,
216 | | },
| |_____- the variant `BuildProductConfig` contains at least 128 bytes
...
638 | ) -> Result<Service> {
| ^^^^^^^^^^^^^^^
|
= help: try reducing the size of `controller::Error`, for example by boxing large elements or replacing it with `Box<controller::Error>`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
= note: `#[warn(clippy::result_large_err)]` on by default
warning: the `Err`-variant returned from this function is very large
--> rust/operator-binary/src/controller.rs:792:6
|
186 | ProductConfigLoadFailed { source: config::Error },
| ------------------------------------------------- the largest variant contains at least 128 bytes
...
210 | BoostrapConfig { source: crate::config::Error },
| ----------------------------------------------- the variant `BoostrapConfig` contains at least 128 bytes
...
213 | / BuildProductConfig {
214 | | source: crate::config::Error,
215 | | rolegroup: RoleGroupRef<NifiCluster>,
216 | | },
| |_____- the variant `BuildProductConfig` contains at least 128 bytes
...
792 | ) -> Result<Service> {
| ^^^^^^^^^^^^^^^
|
= help: try reducing the size of `controller::Error`, for example by boxing large elements or replacing it with `Box<controller::Error>`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
warning: the `Err`-variant returned from this function is very large
--> rust/operator-binary/src/controller.rs:1323:50
|
186 | ProductConfigLoadFailed { source: config::Error },
| ------------------------------------------------- the largest variant contains at least 128 bytes
...
210 | BoostrapConfig { source: crate::config::Error },
| ----------------------------------------------- the variant `BoostrapConfig` contains at least 128 bytes
...
213 | / BuildProductConfig {
214 | | source: crate::config::Error,
215 | | rolegroup: RoleGroupRef<NifiCluster>,
216 | | },
| |_____- the variant `BuildProductConfig` contains at least 128 bytes
...
1323 | fn external_node_port(nifi_service: &Service) -> Result<i32> {
| ^^^^^^^^^^^
|
= help: try reducing the size of `controller::Error`, for example by boxing large elements or replacing it with `Box<controller::Error>`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
warning: `stackable-nifi-operator` (bin "stackable-nifi-operator") generated 3 warnings
- Lingua principale
- Rust
- Stelle
- 48
- Fork
- 12
- Merge medio
- 11h 26m
- PR unite (30g)
- 10
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di stackabletech/nifi-operator
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
stackabletech/nifi-operator#956 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
stackabletech/nifi-operator#879 ·
-
customer-request type/bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
stackabletech/nifi-operator#842 · 1 commento ·
-
size/M type/security
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
stackabletech/nifi-operator#811 · 1 commento ·
-
size/L
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
stackabletech/nifi-operator#810 · 1 commento ·
Tutte le issue di stackabletech/nifi-operator
Issue simili
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
yantrikos/yantrik-os#255 ·
-
Replayed reasoning items send "content": null, which the Responses API schema does not permit Apertabug CLI custom-model
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
raphamorim/rio#1956 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
rust-bitcoin/rust-bitcoin#6930 · 1 commento ·