microcks / microcks/microcks-cli

refactor: remove redundant `strings.HasPrefix` check before `strings.TrimPrefix`

Offen Anfängerfreundlich
#526 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Go
Sterne
52
Forks
68
Ø Merge
6 Std. 54 Min.
Gemergte PRs (30 T.)
10

Beschreibung

### Describe the bug

In `cmd/import.go`, there is an `if strings.HasPrefix(...) { ... = strings.TrimPrefix(...) }` check. This is redundant boilerplate. `strings.TrimPrefix` is safe to call unconditionally because it simply returns the original string if the prefix isn't found.

### Expected behavior

The `strings.TrimPrefix` function should be called without the surrounding `if` statement to reduce cyclomatic complexity.

### Actual behavior

The codebase contains a redundant `if` statement that adds unnecessary lines of code.

### How to Reproduce?

Reproducer:
1. Look at `cmd/import.go`, specifically where path prefixes are trimmed.
2. Observe the `if strings.HasPrefix(...)` check guarding the `strings.TrimPrefix` call.

### Microcks version or git rev

`main` (Latest)

### Install method (`docker-compose`, `helm chart`, `operator`, `docker-desktop extension`,...)

CLI

### Additional information

This is a simple `good first issue` to clean up some Go syntax. Removing the `if` block and unconditionally calling `strings.TrimPrefix` fulfills the same logic cleanly and passes static checkers.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Öffne cmd/import.go und finde die im Issue beschriebene Behandlung des Pfadpräfixes. Ersetze den bedingten Aufruf von strings.TrimPrefix durch einen unbedingten Aufruf; überprüfe anschließend, dass die statischen Prüfer erfolgreich durchlaufen und sich das Verhalten des umgebenden Imports nicht geändert hat.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
go
Bereich
cli
Issue-Typ
Refactoring
Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Aktivitätsstatus
Aktiv
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
92/100

Neue Issues direkt in Ihr Postfach

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