laravel/pint

[BUG] Extension shows "Formatting completed" but silently fails when host PHP version is incompatible

Fermée

#388 ouverte le 5 août 2025

Voir sur GitHubGuide contributeur
 (2 commentaires) (1 réaction) (0 personne assignée)PHP (189 forks)user submission
help wanted

Métriques du dépôt

Stars
 (3 127 étoiles)
Métriques de merge PR
 (Merge moyen 2j 1h) (12 PRs mergées en 30 j)

Description

Pint Version

1.24

PHP Version

7.4

Description

Host PHP: php -v 8.1

  • Project PHP: docker exec [container] php -v 8.2
  • OS: Ubuntu 22.04

Expected Behavior:

  • Extension should warn about PHP version mismatch
  • Formatting should fail visibly (not silently)

Additional Context:

  • Works when manually running ./vendor/bin/pint in the container
  • No errors in VS Code's "Laravel Pint" output panel

Steps To Reproduce

Proposed Fixes:

  1. Add PHP version validation before running Pint
  2. Show explicit errors when:
    • Host PHP < project's required version
    • No PHP binary is found

Guide contributeur

Direction de recherche
Examinez le code de l'extension VS Code dans le dépôt pour comprendre comment la version PHP est déterminée et comment Pint est invoqué. Recherchez les vérifications de version existantes et identifiez où ajouter la validation de la compatibilité de la version PHP de l'hôte. Passez en revue la logique du panneau de sortie de l'extension pour vous assurer que les erreurs sont correctement affichées.
Stack technique
php
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80