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

Improve shop creator with php-cs-fixer #25

Merged
merged 1 commit into from
Nov 10, 2022

Conversation

mflasquin
Copy link
Contributor

@mflasquin mflasquin commented Nov 2, 2022

Questions Answers
Description? Improve shop creator with setup of github workflow and add php-cs-fixer
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #23
How to test? see mflasquin#1
Possible impacts? Please indicate what parts of the software we need to check to make sure everything is alright.

nicosomb
nicosomb previously approved these changes Nov 2, 2022
Copy link

@nicosomb nicosomb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great improvement!

@mflasquin
Copy link
Contributor Author

POC pipeline : mflasquin#1

@mflasquin mflasquin force-pushed the improve/23-php-cs-fixer branch from 9d825e9 to 708953d Compare November 10, 2022 08:27
@mflasquin
Copy link
Contributor Author

Hi @matks, this PR is also rebased 😄 so if you want to take a look you're welcome

@mflasquin mflasquin requested a review from nicosomb November 10, 2022 08:40
@matks
Copy link
Contributor

matks commented Nov 10, 2022

Thank you @mflasquin

@matks matks merged commit 941084d into PrestaShop:master Nov 10, 2022
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.

Improve this repository, add tests
3 participants