-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Correct spelling #19684
Correct spelling #19684
Conversation
Hi @ravi-chandra3197. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @VladimirZaets, thank you for the review. |
@ravi-chandra3197 please do not produce useless merge commits. |
Hello @orlangur |
@ravi-chandra3197 sure, here is simple way: https://coderwall.com/p/ysor6a/git-merge-squash-for-merging-without-commit-history |
@orlangur Thanks for the response but my concern is how to remove commit from an existing branch or rewrite branch history. |
@ravi-chandra3197 simply squash all changes into single commit and force push as provided article shows. |
214efed
to
37db806
Compare
Hello, @orlangur I have removed merge commits. can review it. |
@ravi-chandra3197 very nice, thank you! 👍 |
Hi @ravi-chandra3197. Thank you for your contribution. |
Description (*)
correct spelling
Fixed Issues (if relevant)
Manual testing scenarios (*)
N/A
Contribution checklist (*)