apache / apache/datafusion

Suggestion: enable `unreachable_pub` lint at workspace level

Aperta
#22,154 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Rust
Stelle
9.3k
Fork
2.4k
Merge medio
3g 11h
PR unite (30g)
360

Descrizione

Why looking at the code I thought I could use `InProgressSpillFile` since it marked as `pub`, or the function `create_in_progress_file` that is also `pub`, but `InProgressSpillFile` mod is behind `pub(crate)` so to avoid confusion when reading the source code

I want to enable `unreachable_pub` lint at the workspace level so this won't happen since you will be forced to specify the correct visibility

Applying this can be done using: `cargo fix --workspace --all-targets`

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia dalla configurazione di Cargo a livello di workspace e verifica come sono attualmente definite le impostazioni dei lint. Esegui `cargo fix --workspace --all-targets`, esamina le modifiche alla visibilità proposte e verifica che il workspace venga compilato e che il lint `unreachable_pub` sia abilitato senza avvisi residui.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
rust
Ambito
build-system, tooling
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
68/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.