-
Notifications
You must be signed in to change notification settings - Fork 9.4k
MFTF: Extract Action Groups to separate files - magento/module-reports #25812
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
MFTF: Extract Action Groups to separate files - magento/module-reports #25812
Conversation
Hi @lbajsarowicz. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
Hi @ihor-sviziev, thank you for the review. |
QA not applicable |
@lbajsarowicz |
@ihor-sviziev Definitely after cleaning up the mess that we already have - we should have such. This should also check if the filename corresponds with the only test name inside. |
@lbajsarowicz could you report an issue for that, just in order not to miss it |
Yes, I'll do it for both |
Thank you!
…On Fri, 29 Nov 2019 at 13:40, Lukasz Bajsarowicz ***@***.***> wrote:
Yes, I'll do it for both magento2 repository and
magento-functional-tests-migration
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#25812?email_source=notifications&email_token=AAOJOUKDODHNJQ4HYSOOVOLQWD5UHA5CNFSM4JSWNRX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFOU5OY#issuecomment-559763131>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAOJOUMGH7IWOONF3IJNUVLQWD5UHANCNFSM4JSWNRXQ>
.
|
@VladimirZaets Could you explain why this Pull Request cannot be merged to 2.3? There are no backward incompatible changes. |
Hi @lbajsarowicz, thank you for your contribution! |
Description (*)
Extract each Action Group to separate file, to follow MFTF Best Practices.
Fixed Issues (if relevant)
Questions or comments
Had to extract the changes per-module. The previous try failed, because of conflicts.
Contribution checklist (*)