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
Refactor transformations #905
Conversation
* | ||
* @param string $suiteName | ||
* @param string $pattern | ||
* @param string$language |
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.
Missing space.
|
||
use Behat\Behat\Transformation\Transformation; | ||
use Behat\Testwork\Call\RuntimeCallee; | ||
|
||
/** | ||
* Transformation that is created and executed in the runtime. | ||
* | ||
* @deprecated Will be removed in 4.0. Use specific transformations instead |
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.
I suggest using the @trigger_error('...', E_USER_DEPRECATED)
convention used by Symfony (which reminds me that someone wrote a Behat extension to collect deprecations, which I said I would review and I forgot to do it)
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.
btw, if you rename the class, deprecating it is useless: it is already a BC break. you would need to keep it in the Call
namespace
I'll move |
23a5cc4
to
834e78c
Compare
This PR extracts individual transformations from
RepositoryArgumentTransformer
into individual classes underBehat\Behat\Transformation\Transformation
namespace. This should make adding new transformations and fixing old ones a breeze.This PR is basis for upcoming transformation improvements in 3.2.