-
Notifications
You must be signed in to change notification settings - Fork 9.4k
[Forwardport] Updated SynonymGroup.xml #16635
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
[Forwardport] Updated SynonymGroup.xml #16635
Conversation
…Setup module's scan function.
Hi @sanganinamrata. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
@magento-engcom-team combine 16636 16637 |
Hi @sidolov. Thank you for your request. I'm working on combining the pull requests for you |
@sanganinamrata all pull requests have been successfully combined together:
|
Hi @orlangur, thank you for the review. |
Hi @sanganinamrata. Thank you for your contribution. |
Original Pull Requests
#16557
#16636
#16637
Description
Fixed Issues (if relevant)
Updated SynonymGroup.xml by removing extra space from the value of is_required XML node.
Corrected function comment for Magento Setup module includeClasses function.
Removed double occurrences from jQuery, angular JS files and Magento Setup module's scan function's comment.
Manual testing scenarios
N/A
Contribution checklist