aws / aws/aws-codeguru-cli

`--commit-range` doesn't restrict the recommendations to changed files in that range

Offen
#35 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
23
Forks
13
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

### Issue
When I run the cli command to get a security scan for a given commit range, it includes recommendations across the entire package rather than just the specified commit range.

### Steps to reproduce
1. Run `aws-codeguru-cli --root-dir . --profile --mode security --commit-range HEAD^:HEAD`
2. Observe the the file at `code-guru/recommendations.json` includes recommendations across your entire package.

### Expected Result
Only include recommendations that are part of the diff within the commit range.

### Notes
I tried removing the commit range and noticed that the output is identical which makes me think that the flag is ignored.

This seems related to #9 but I wasn't able to validate the output of the jobs mentioned in that query.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Führen Sie den gemeldeten aws-codeguru-cli-Befehl mit und ohne --commit-range aus und vergleichen Sie code-guru/recommendations.json. Verfolgen Sie den Einstiegspunkt von --commit-range und prüfen Sie das zugehörige Issue #9; als abgeschlossen gilt die Aufgabe, wenn recommendations.json nur Empfehlungen für Dateien enthält, die im angegebenen Commit-Bereich geändert wurden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
cli
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.