-
Notifications
You must be signed in to change notification settings - Fork 45
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
Update README.md #34
Update README.md #34
Conversation
Co-authored-by: Andre_601 <[email protected]>
Co-authored-by: Andre_601 <[email protected]>
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.
Almost good. Just needs some changes in some wording imo.
Co-authored-by: Andre_601 <[email protected]>
I just thought, I will squash and merge when you approve this pr, but anyway why do you not use squash and merge |
I thought you were only waiting for my review as I was late at giving review. So I made sure it gets merged immediately |
Squash and merge will make one commit, and that's the best I consider when there are more than 10 commits in pr. |
I'll keep that in mind. |
Ok, good night |
Thank you for contributing! Please check the following things before submiting your PR:
Required:
If necessary:
Closes #
Note: we will close your PR without comment if you do not check the required boxes above and provide ALL requested information.