Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dodanie sprawdzania poprawności kodu php #498

Merged
merged 4 commits into from
Jun 2, 2024

Conversation

wojsmol
Copy link
Contributor

@wojsmol wojsmol commented Apr 13, 2024

No description provided.

@wojsmol wojsmol requested a review from artur-stepien April 13, 2024 20:53
@wojsmol wojsmol force-pushed the add/php-syntax-check branch from a50c4ee to e06692e Compare April 13, 2024 21:05
@wojsmol wojsmol force-pushed the add/php-syntax-check branch from e06692e to b3e3e2c Compare April 13, 2024 21:31
@artur-stepien
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor Author

wojsmol commented Apr 20, 2024

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
Copy link
Contributor

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.

@artur-stepien artur-stepien merged commit 3c9fe0b into 5.1-dev Jun 2, 2024
2 checks passed
@wojsmol wojsmol deleted the add/php-syntax-check branch July 8, 2024 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants