-
-
Notifications
You must be signed in to change notification settings - Fork 112
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 #69
Update Readme.md #69
Conversation
@PreciousOritsedere I think it's best you wait for PR #68 to be closed so we don't have duplicates |
I think @possumbilities or @TimidRobot is in the best position to review both and determine which one to close. |
… and removing line inconsistency
This PR is already out of the scope of the linked issue |
@ImaJin14 how please? Can you explain further |
Your issue states improvement on readme.md file but your commits show that you've changed more than just the readme.md file I suggest you revert changes and resolve just the readme.md adjustments |
I was actually attempting to create another PR. But I think the commit merged into this. Because I couldn’t see the option to create a different Pr. I think the first one has to be merged first before I can create again |
@PreciousOritsedere I'm grateful for your excitement to get started on this, but I'm going to close this PR since it's got extra changes not relevant to the Issue, and suggest you wait to try again until the original Issue #68 is fully approved. |
Fixes
Fixes #68 by @PreciousOritsedere
Description
Added more distinctive instructions to the readme.md
Technical details
Tests
Screenshots
Checklist
Update index.md
).main
ormaster
).visible errors.
Developer Certificate of Origin
For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."
Developer Certificate of Origin