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

fix: CI #21

Merged
merged 7 commits into from
Jun 21, 2024
Merged

fix: CI #21

merged 7 commits into from
Jun 21, 2024

Conversation

Jean-Beru
Copy link
Contributor

@Jean-Beru Jean-Beru commented Mar 6, 2024

Improve CI:

  • split job between
    • php-cs-fixer
    • phpstan
    • phpunit
  • configure phpunit matrix
    • Symfony 7 / PHP 8.2 and 8.3 / highest and lowest deps
    • Symfony LTS / PHP 8.2 / highest deps
  • triggered when
    • pull request on main
    • push on main (after merge)
    • scheduled daily to check dependencies updates compatibility

Just one minor detail... It is only tested locally using nektios/act since this repository is still private.

Fixes #53

@Jean-Beru Jean-Beru force-pushed the fix/ci branch 2 times, most recently from f018734 to 5c995d6 Compare March 6, 2024 20:34
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
@smnandre
Copy link
Contributor

Just one minor detail... It is only tested locally using nektios/act since this repository is still private.

You could add a "dispatch" trigger to manually test if needed

@Jean-Beru Jean-Beru self-assigned this May 27, 2024
@Jean-Beru Jean-Beru removed this from the v0.1.0 milestone Jun 5, 2024
@Jean-Beru Jean-Beru force-pushed the fix/ci branch 2 times, most recently from 824e99d to 534209e Compare June 6, 2024 08:05
@Jean-Beru Jean-Beru merged commit 75c820a into main Jun 21, 2024
@Jean-Beru Jean-Beru deleted the fix/ci branch June 21, 2024 12:59
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.

Add CI
3 participants