-
Notifications
You must be signed in to change notification settings - Fork 3
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
updated readme #22
updated readme #22
Conversation
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.
Hi @pojith,
thanks for contributing to #20! I thought you wanted to start with #19 as you commented directly on this issue, and it is a good first issue for someone not familiar with the repository.
That being said, any help is appreciated, I made a couple of comments regarding your contribution here that would need to be addressed before merging your PR. Let me know if something needs to be further explained.
Best,
README.md
Outdated
<a rel="license" href="http://creativecommons.org/licenses/by/4.0/">Creative | ||
Commons Attribution 4.0 International License</a>. | ||
This work is licensed under a | ||
Creative |
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.
Creative |
put on a single line
README.md
Outdated
Commons Attribution 4.0 International License</a>. | ||
This work is licensed under a | ||
Creative | ||
Commons Attribution 4.0 International License. |
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.
Commons Attribution 4.0 International License. |
put on a single line
README.md
Outdated
@@ -703,9 +703,6 @@ Tatusova, et al. 2008. “The Minimum Information about a Genome Sequence | |||
“GSC MIxS: Host-associatedMIMS.” | |||
<https://genomicsstandardsconsortium.github.io/mixs/Host-associatedMIMS/>. | |||
|
|||
“GSC MIxS: Human-associatedMIMS.” |
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.
Is there any reason why you removed this URL @pojith?
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.
no
Update README.md
can you check now . |
<br><br> This work is licensed under a | ||
<a rel="license" href="http://creativecommons.org/licenses/by/4.0/">Creative | ||
Commons Attribution 4.0 International License</a>. | ||
+This work is licensed under a [Creative Commons Attribution 4.0 International License](http://creativecommons.org/licenses/by/4.0/). |
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.
Thanks! I removed the extra lines that must have slipped from a copy paste.
No description provided.