FuelLabs/sway

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

Open

#4 451 ouverte le 17 avr. 2023

Voir sur GitHub
 (4 commentaires) (0 réactions) (1 assigné)Rust (5 421 forks)batch import
ODHack13enhancementforcforc-pkggood first issuemay be rewarded

Métriques du dépôt

Stars
 (61 702 stars)
Métriques de merge PR
 (Merge moyen 5j 4h) (19 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez comment cargo génère des avertissements pour les paquets non listés dans les membres de l'espace de travail (par exemple, le message d'avertissement concernant une configuration d'espace de travail invalide). Ensuite, localisez le code de gestion de l'espace de travail dans sway, probablement dans sway core ou des crates similaires. Ajoutez un avertissement similaire lorsqu'un nouveau paquet est créé dans un répertoire de l'espace de travail mais n'est pas inclus dans les membres de [workspace] dans le Cargo.toml racine. Suivez les modèles d'avertissement existants et assurez la compatibilité avec l'analyse du manifeste de l'espace de travail.
Stack technique
rust
Domaine
tooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75