Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

`vp migrate --full` asks to set up pre-commit hooks even when they're already configured

Geschlossen Anfängerfreundlich
  • Stimmt etwas nicht?
#3,096 2 Kommentare 3 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

@0froq arbeitet bereits daran.

Seit 19.10.2026.

  • #3118 von @0froq — offen

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
74/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
rust
Bereich
cli, developer-experience

Rechercherichtung

Lies zuerst promptGitHooks() und die vorhandene hasExistingViteHooksPolicy()-Prüfung, die von setupGitHooks() verwendet wird. Verfolge, wie vp migrate --full mit vorhandenen Hooks umgeht, und suche nach den relevanten Prompt-Tests. Fertig ist die Aufgabe, wenn vorhandene Vite+-Hooks erkannt werden und der Benutzer nicht mehr gefragt wird, sie einzurichten, als wären sie nicht vorhanden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

contribution welcome

Description

In a project that already has Vite+ hooks (.vite-hooks/pre-commit, core.hooksPath=.vite-hooks/_), vp migrate --full still asks:

◇ Set up pre-commit hooks to run formatting, linting, and type checking with auto-fixes?
  Yes / No

It defaults to Yes and doesn't say hooks already exist. Answering No sounds like it might disable them, though it doesn't.

The agent prompt in the same run detects the existing file and offers Append/Skip:

› Agent instructions already exist at AGENTS.md.
    Append
  › Skip: Leave existing file unchanged

promptGitHooks() doesn't check hasExistingViteHooksPolicy(), which setupGitHooks() already uses to leave the hook as it is.

Suggested solution

When existing Vite+ hooks are detected, either:

  • Skip the prompt and log Git hooks already configured, or
  • Say so in the prompt and offer Keep/Skip, like the agent prompt does.

Additional context

Validations

  • Read the Contributing Guidelines.
  • Confirm this request is for Vite+ itself and not for Vite, Vitest, tsdown, Rolldown, or Oxc.
  • Check that there isn't already an issue requesting the same feature.
Vorherrschende Sprache
Rust
Sterne
6.5k
Forks
296
Ø Merge
1 T. 9 Std.
Gemergte PRs (30 T.)
224

Entwicklungsumgebung

In Codespaces öffnen

Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus voidzero-dev/vite-plus

Alle Issues in voidzero-dev/vite-plus

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

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