Skip to content

nucleotide-count: Remove reference to RNA from description#918

Merged
Insti merged 2 commits intoexercism:masterfrom
Insti:Nucleotide_count
Oct 2, 2017
Merged

nucleotide-count: Remove reference to RNA from description#918
Insti merged 2 commits intoexercism:masterfrom
Insti:Nucleotide_count

Conversation

@Insti
Copy link
Copy Markdown
Contributor

@Insti Insti commented Oct 1, 2017

In #913 the description of DNA was made more scientifically accurate.

However the reference to RNA is not relevant to this problem and is likely to cause confusion, so I have removed it from the description.

The reference to RNA is not relevant to this problem and will only cause
confusion.
@Insti Insti requested a review from rpottsoh October 1, 2017 18:48
Copy link
Copy Markdown
Member

@rpottsoh rpottsoh left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Copy Markdown
Member

@petertseng petertseng left a comment

Choose a reason for hiding this comment

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

I am thinking you might want to also remove line 12 which is now a second trailing newline

@Insti Insti merged commit 37ee0f4 into exercism:master Oct 2, 2017
@Insti Insti deleted the Nucleotide_count branch October 2, 2017 19:08
@Insti
Copy link
Copy Markdown
Contributor Author

Insti commented Oct 2, 2017

Thanks for the reviews! ❤️

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