FuelLabs/sway

Warn users if they create a package in a workspace without listing them as a member

Open

#4451 aperta il 17 apr 2023

Vedi su GitHub
 (4 commenti) (0 reazioni) (1 assegnatario)Rust (5421 fork)batch import
ODHack13enhancementforcforc-pkggood first issuemay be rewarded

Metriche repository

Star
 (61.702 star)
Metriche merge PR
 (Merge medio 22h 6m) (16 PR mergiate in 30 g)

Descrizione

cargo warns users with a warning if they create a package inside a workspace directory like the following:

warning: compiling this new package may not work due to invalid workspace configuration

current package believes it's in a workspace when it's not:
current:   /Users/kayagokalp/fuel/dev/sway/forc-error/Cargo.toml
workspace: /Users/kayagokalp/fuel/dev/sway/Cargo.toml

We might want to do something similar. In our case a child package wouldn't be "thinking" it's in a workspace but it is still pretty easy to forget listing a package in the workspace manifest file.

Guida contributor

Direzione di ricerca
Indaga come cargo genera avvisi per i pacchetti non elencati nei membri dell'area di lavoro (ad esempio, il messaggio di avviso sulla configurazione non valida dell'area di lavoro). Quindi, individua il codice di gestione dell'area di lavoro in sway, probabilmente in sway core o in crate simili. Aggiungi un avviso simile quando un nuovo pacchetto viene creato all'interno di una directory dell'area di lavoro ma non incluso nei membri di [workspace] nel Cargo.toml radice. Segui i modelli di avviso esistenti e assicura la compatibilità con l'analisi del manifesto dell'area di lavoro.
Tech stack
rust
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75