rome/tools

đź“Ž New `--changed` argument

Ouverte

#4 453 ouverte le 9 mai 2023

Voir sur GitHubGuide contributeur
 (2 commentaires) (2 réactions) (0 personne assignée)Rust (647 forks)batch import
A-CLIA-ProjectHelp wantedI-Normal

Métriques du dépôt

Stars
 (23 438 étoiles)
MĂ©triques de merge PR
 (Aucune PR mergée en 30 j)

Description

Description

Rome now has support for VCS, which means that we can unlock new features that make the life of developers easier!

One of these features is the addition of a new argument called --changed to the CLI. When we pass this argument, Rome will execute check/format/etc. only to those files that were changed.

Technical details

I am still determining the implementation of this argument. There are two alternatives:

  1. execute the command via Command, we pass the proper arguments to the git program, and we read the output that was generated. Although I don't know yet what's the correct command/arguments to get an accurate output
  2. Use the crate git2. While this crate is phenomenal, its size is not small (207kb)

What do you guys think?

Guide contributeur

Direction de recherche
Explorez comment obtenir les fichiers modifiés à partir de Git (git diff name only) ou évaluez l'impact de la taille de la crate git2. Envisagez d'implémenter d'abord avec Command pour éviter le gonflement des dépendances.
Stack technique
rust
Domaine
clideveloper experience
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.
3
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.
PlutĂ´t claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50