Skip to content

fixed typos for modfinder chapter#19

Merged
lafita merged 2 commits intobiojava:masterfrom
jjgao:modfinder-typo-fixes
Feb 14, 2017
Merged

fixed typos for modfinder chapter#19
lafita merged 2 commits intobiojava:masterfrom
jjgao:modfinder-typo-fixes

Conversation

@jjgao
Copy link
Copy Markdown
Member

@jjgao jjgao commented Jan 23, 2017

No description provided.

@jjgao jjgao requested review from lafita and pwrose January 23, 2017 20:16
@jjgao
Copy link
Copy Markdown
Member Author

jjgao commented Feb 6, 2017

@lafita @pwrose : would you please review the merge the pull request? Thanks.

Comment thread modfinder/add-protein-modification.md Outdated
@@ -1,4 +1,4 @@
How to define a new protein modiifcation?
How to define a new protein modifcation?
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: should be modification

Comment thread modfinder/add-protein-modification.md Outdated
[Home](../README.md)
| [Book 6: The ModFinder Modules](README.md)
| Chapter 4 - How to define a new protein modiifcation
| Chapter 4 - How to define a new protein modifcation
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same typo again

Copy link
Copy Markdown
Member

@lafita lafita left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jjgao sorry, I thought that the PR was already merged.

Prev: [Chapter 2 : How to get a list of supported protein modifications](supported-protein-modifications.md)

Next: [Chapter 4 : How to define a new protein modiifcation](add-protein-modification.md)
Next: [Chapter 4 : How to define a new protein modifcation](add-protein-modification.md)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same typo that @andreasprlic described.

@lafita lafita merged commit 42419ae into biojava:master Feb 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants