Create two separate hook libraries: folder-copy, and config-manager

Offen
  • Stimmt etwas nicht?
#6 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
35/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
php
Bereich
tooling

Rechercherichtung

Beginne beim Installer und untersuche den bestehenden Code zur Verwaltung von PHPCS installed_paths; prüfe anschließend, wie phpcs-coding-standard und postInstall-Hooks behandelt werden. Trenne die Ansätze copy-folders und manage-config hinter einer öffentlichen Schnittstelle, verwende den extra-Eintrag in composer.json als Umschalter und verifiziere, dass beide Installer-Pfade die ausgewählte Methode berücksichtigen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement help wanted

There are two different ways to manage PHPCS coding standards installation:

  • You can copy the standards folder directly into phpcs's installed path, so they are just "there" for phpcs to use.
  • You can actively manage the entries in phpcs's configuration file for the installed_paths directive, essentially informing phpcs where to find your standards in their "native" composer-installed locations.

The first method is the currently-implemented approach, but the code already exists to perform the second approach as well.

What needs to happen is this:

  • Separate the two approachs out into two different classes, but that share the same public interface (making them interchangeable).
  • Update the installer to check for an extra entry in the project's composer.json file that can be used as a switch to determine which method is preferred by the project (copy-folders or manage-config).
  • Update the installer to use the preferred method. This should apply to both the phpcs-coding-standard type as well as the postInstall hooks.
Vorherrschende Sprache
PHP
Sterne
0
Forks
1
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

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 loadsys/composer-plugins

Alle Issues in loadsys/composer-plugins

Ähnliche Issues

Weitere Issues zu PHP

Neue Issues direkt in Ihr Postfach

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