Dodanie sprawdzania poprawności kodu php

[artur-stepien]

Joomla! 5 ma minimalne wymaganie co do PHP na 8.1, nie 8.3 więc pewnie najlepiej sprawdzać 8.1-8.3. Kolejna sprawa to obecna konfiguracja sprawdza zgodność dla wszystkiego z tego co widzę (także skryptów budujących paczkę i vendora)

[artur-stepien]

Joomla! 5 ma minimalne wymaganie co do PHP na 8.1, nie 8.3 więc pewnie najlepiej sprawdzać 8.1-8.3. Kolejna sprawa to obecna konfiguracja sprawdza zgodność dla wszystkiego z tego co widzę (także skryptów budujących paczkę i vendora)

@wojsmol Jest jeszcze jedna sprawa. Dobrze żeby test był dostępny po stronie klienta także. Czyli trzeba by to wczepić w proces budowania i testowania paczki, którzy można uruchomić lokalnie i po stronie githuba też. Więc ja bym był za tym, żeby to wrzucić jako issue żeby się nie zgubiło i dodam to razem do procesu budowania paczki.

[wojsmol]

Joomla! 5 ma minimalne wymaganie co do PHP na 8.1, nie 8.3 więc pewnie najlepiej sprawdzać 8.1-8.3. Kolejna sprawa to obecna konfiguracja sprawdza zgodność dla wszystkiego z tego co widzę (także skryptów budujących paczkę i vendora)

@wojsmol Jest jeszcze jedna sprawa. Dobrze żeby test był dostępny po stronie klienta także. Czyli trzeba by to wczepić w proces budowania i testowania paczki, którzy można uruchomić lokalnie i po stronie githuba też. Więc ja bym był za tym, żeby to wrzucić jako issue żeby się nie zgubiło i dodam to razem do procesu budowania paczki.

Dodałem sprawdzanie dla php 8.1 i 8.2 używana w tej chwili akcja nie wspiera jeszcze php 8.3 tak samo jak wykluczania folderów. IMHO lepiej aby byłó sprawdzanie nawet tylko na GitHubie niż wcale nie mieć sprawdzania

[artur-stepien]

Joomla! 5 ma minimalne wymaganie co do PHP na 8.1, nie 8.3 więc pewnie najlepiej sprawdzać 8.1-8.3. Kolejna sprawa to obecna konfiguracja sprawdza zgodność dla wszystkiego z tego co widzę (także skryptów budujących paczkę i vendora)

@wojsmol Jest jeszcze jedna sprawa. Dobrze żeby test był dostępny po stronie klienta także. Czyli trzeba by to wczepić w proces budowania i testowania paczki, którzy można uruchomić lokalnie i po stronie githuba też. Więc ja bym był za tym, żeby to wrzucić jako issue żeby się nie zgubiło i dodam to razem do procesu budowania paczki.

Dodałem sprawdzanie dla php 8.1 i 8.2 używana w tej chwili akcja nie wspiera jeszcze php 8.3 tak samo jak wykluczania folderów. IMHO lepiej aby byłó sprawdzanie nawet tylko na GitHubie niż wcale nie mieć sprawdzania

Przepisać budowanie paczki i dodać testowanie pewnie dopiero będę mógł pod koniec maja lub początek czerwca patrząc po obecnym kalendarzu prac także jak tylko sprawdzę to wprowadzimy i najwyżej jak będzie gotowe pełniejsze rozwiązanie się zamieni.