Skip to content

Convert Behat annotations to PHP attributes#18810

Closed
loic425 wants to merge 1 commit intoSylius:2.3from
loic425:chore/behat-annotations-to-attributes
Closed

Convert Behat annotations to PHP attributes#18810
loic425 wants to merge 1 commit intoSylius:2.3from
loic425:chore/behat-annotations-to-attributes

Conversation

@loic425
Copy link
Copy Markdown
Member

@loic425 loic425 commented Feb 11, 2026

Q A
Branch? 2.3
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

@loic425 loic425 requested review from a team as code owners February 11, 2026 07:44
@loic425 loic425 added the Maintenance CI configurations, READMEs, releases, etc. label Feb 11, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 11, 2026

Important

Review skipped

Too many files!

This PR contains 214 files, which is 64 over the limit of 150.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@loic425 loic425 added the Behat Issues and PRs aimed at improving Behat usage. label Feb 11, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 11, 2026

❌ Preview Environment deleted from Bunnyshell

Available commands:

  • 🚀 /bns:deploy to redeploy the environment

@loic425 loic425 force-pushed the chore/behat-annotations-to-attributes branch from c371e8e to bd118db Compare February 11, 2026 08:11
@loic425 loic425 force-pushed the chore/behat-annotations-to-attributes branch from bd118db to 42bd64a Compare February 11, 2026 08:12
@loic425 loic425 marked this pull request as draft February 11, 2026 08:17
@loic425
Copy link
Copy Markdown
Member Author

loic425 commented Feb 11, 2026

Closed in favor of #18811

@loic425 loic425 closed this Feb 11, 2026
@loic425 loic425 deleted the chore/behat-annotations-to-attributes branch February 11, 2026 08:30
TheMilek added a commit that referenced this pull request Feb 11, 2026
| Q               | A
|-----------------|-----
| Branch?         | 2.3
| Bug fix?        | no
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | 
| License         | MIT

<!--
 - Bug fixes must be submitted against the 1.14 or 2.1 branch
 - Features and deprecations must be submitted against the 2.2 branch
 - Make sure that the correct base branch is set

To be sure you are not breaking any Backward Compatibilities, check the
documentation:

https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html
-->

Smaller version of #18810 only applied on Domain contexts.


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Refactor**
* Modernized Behat test integration by converting step-definition
annotations from docblock style to native PHP 8 attributes across test
contexts.
* Updated tooling configuration to enable Behat-style attribute handling
during automated code transformations.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Behat Issues and PRs aimed at improving Behat usage. Maintenance CI configurations, READMEs, releases, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant