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

Devtools #88

Merged
merged 5 commits into from
Jun 2, 2020
Merged

Devtools #88

merged 5 commits into from
Jun 2, 2020

Conversation

Matt75
Copy link
Contributor

@Matt75 Matt75 commented May 28, 2020

  1. Add php-dev-tools
  2. Add Github Actions
  3. Run header-stamp
  4. Run php-cs-fixer
  5. Run phpstan
  6. Fix phpstan feedbacks

@Matt75 Matt75 marked this pull request as draft May 28, 2020 15:03
@Matt75 Matt75 requested a review from Quetzacoalt91 May 28, 2020 15:06
@Matt75 Matt75 force-pushed the devtools branch 5 times, most recently from 60f6973 to c7bf9d8 Compare May 28, 2020 18:39
@Matt75 Matt75 marked this pull request as ready for review May 28, 2020 18:41
@Matt75 Matt75 requested a review from PierreRambaud May 28, 2020 18:41
@@ -0,0 +1,61 @@
name: Build & Release draft
Copy link
Contributor

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 ?

Copy link
Member

@Quetzacoalt91 Quetzacoalt91 May 29, 2020

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

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 ?

Copy link
Contributor Author

@Matt75 Matt75 May 29, 2020

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

Copy link
Contributor

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']

Copy link
Contributor Author

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.

@Matt75 Matt75 requested review from matks and a team May 29, 2020 08:32
Copy link
Member

@Quetzacoalt91 Quetzacoalt91 left a 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!

.prettyci.composer.json Outdated Show resolved Hide resolved
@Matt75 Matt75 merged commit bcd57b6 into PrestaShop:dev Jun 2, 2020
@Matt75 Matt75 deleted the devtools branch June 2, 2020 11:08
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.

3 participants