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

[ECS] Add Code Checking In the Tests Directory and Implement Fixes For the New Code #15558

Merged
merged 5 commits into from Dec 5, 2023

Conversation

Rafikooo
Copy link
Contributor

@Rafikooo Rafikooo commented Nov 23, 2023

Q A
Branch? 1.13
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets #15532 (comment)
License MIT

To save developers' effort during the code review, we added coding standards checks into the unit tests directory as well.

@Rafikooo Rafikooo added the DX Issues and PRs aimed at improving Developer eXperience. label Nov 23, 2023
@Rafikooo Rafikooo requested a review from a team as a code owner November 23, 2023 18:22
Copy link

github-actions bot commented Nov 23, 2023

Bunnyshell Preview Environment deleted

Available commands:

  • /bns:deploy to redeploy the environment

diimpp
diimpp previously approved these changes Nov 24, 2023
Wojdylak
Wojdylak previously approved these changes Nov 24, 2023
@jakubtobiasz
Copy link
Member

@Rafikooo can you solve conflicts 😅?

@Rafikooo Rafikooo dismissed stale reviews from Wojdylak and diimpp via c04fa67 November 29, 2023 16:12
@Rafikooo Rafikooo force-pushed the ecs-in-tests branch 2 times, most recently from c04fa67 to 86f6e2b Compare November 29, 2023 19:42
@probot-autolabeler probot-autolabeler bot added the Maintenance CI configurations, READMEs, releases, etc. label Nov 29, 2023
@probot-autolabeler probot-autolabeler bot added the API APIs related issues and PRs. label Nov 29, 2023
@probot-autolabeler probot-autolabeler bot added the Admin AdminBundle related issues and PRs. label Nov 29, 2023
@Rafikooo Rafikooo force-pushed the ecs-in-tests branch 2 times, most recently from 1e3f022 to 152c954 Compare November 30, 2023 10:35
NoResponseMate
NoResponseMate previously approved these changes Nov 30, 2023
phpstan.neon.dist Outdated Show resolved Hide resolved
psalm.xml Outdated Show resolved Hide resolved
Wojdylak
Wojdylak previously approved these changes Nov 30, 2023
vvasiloi
vvasiloi previously approved these changes Dec 1, 2023
@Rafikooo Rafikooo changed the title [ECS] Add checking in the tests directory [ECS] Add Code Checking In the Tests Directory and Implement Fixes For the New Code Dec 5, 2023
@NoResponseMate NoResponseMate merged commit 812926f into Sylius:1.13 Dec 5, 2023
25 checks passed
@NoResponseMate
Copy link
Contributor

Thank you, @Rafikooo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin AdminBundle related issues and PRs. API APIs related issues and PRs. DX Issues and PRs aimed at improving Developer eXperience. Maintenance CI configurations, READMEs, releases, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants