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

B 7.0.x use light workflow #30

Open
wants to merge 23 commits into
base: b-7.0.x
Choose a base branch
from
Open

Conversation

joernott
Copy link

This PR switches the github actions to the univerasal light workflow.
There are a few things we should clarify fefore merging:

  • I have noticed that phpcs was installed in composer.json but no tests/phpcs.xml was present. I've added that
  • I also added configs for phpmd/phpstan and a 2nd test plan which also tests phpcs, phpmd and phpstan. We should discuss whether you want those tests to run and if you want them installed as part of require-dev in composer.json. Apart from phpstan reporting 81 errors which are "ignorable", they look OK. If you don't want those, I should remove the configs before merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant