-
Notifications
You must be signed in to change notification settings - Fork 72
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
Devtools #88
Devtools #88
Conversation
Matt75
commented
May 28, 2020
•
edited
Loading
edited
- Add php-dev-tools
- Add Github Actions
- Run header-stamp
- Run php-cs-fixer
- Run phpstan
- Fix phpstan feedbacks
60f6973
to
c7bf9d8
Compare
@@ -0,0 +1,61 @@ | |||
name: Build & Release draft |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Having release drafter AND this GA will be OK ? They will not fight each other ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is actually how we should trigger release drafter from now. They replaces their application with a GitHub Action
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
presta-versions: ['1.7.0.3', 'latest'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So this GA will go through all PS versions from 1.7.0.3 to latest ?
It's not too long ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it's only running on provided docker tag : so only for tag 1.7.0.3
and tag latest
😉
4 completed jobs in 1m 57s
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah ok 😄
So maybe it would be interesting to run it against the 3 latest versions as these are the "most used" versions ?
=> presta-versions: ['1.7.0.3', '1.7.5.2', '1.7.6.5', 'latest']
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We running PHPStan only if we know breaking changes that can potentially broken module.
There were several between 1.7.0.3
and latest
but no need to test others versions for this module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 unneeded file that can be deleted. Otherwise, all good!