rome/tools

📎 New `--changed` argument

Offen

#4.453 geöffnet am 09.05.2023

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)Rust (647 Forks)batch import
A-CLIA-ProjectHelp wantedI-Normal

Repository-Metriken

Stars
 (23.438 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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?

Contributor Guide

Research-Richtung
Erkunde, wie man geänderte Dateien von Git abruft (git diff name only) oder bewerte die Größenauswirkungen der git2 Crate. Ziehe in Betracht, zuerst mit Command zu implementieren, um Aufblähung der Abhängigkeiten zu vermeiden.
Tech Stack
rust
Domain
clideveloper experience
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50