Skip to content

sum-of-multiples: Removed a duplicated sentence in README#358

Merged
petertseng merged 1 commit intoexercism:masterfrom
GoneUp:typo
Oct 10, 2017
Merged

sum-of-multiples: Removed a duplicated sentence in README#358
petertseng merged 1 commit intoexercism:masterfrom
GoneUp:typo

Conversation

@GoneUp
Copy link
Copy Markdown
Contributor

@GoneUp GoneUp commented Oct 7, 2017

Basically what the title says. I stumbled across the duplicated sentence while doing the exercises.

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.

thank you, this is good (Because the sentence was indeed duplicated).

I need to ask you a favour. I need you to make the same PR to https://github.com/exercism/problem-specifications/blob/master/exercises/sum-of-multiples/description.md . If you do not, then when this track regenerates its README using that file, the change you've made here will get overwritten. We wouldn't want that to happen.

@GoneUp
Copy link
Copy Markdown
Contributor Author

GoneUp commented Oct 10, 2017

Ah, okay. I didn't know that all tracks have (at least partly) the same description. I will do that tomorrow.

@ijanos
Copy link
Copy Markdown
Contributor

ijanos commented Oct 10, 2017

Thank you! I'm closing this one now, if you don't mind.

@petertseng
Copy link
Copy Markdown
Member

OK, it got merged in problem-specifications.

@petertseng petertseng reopened this Oct 10, 2017
@petertseng
Copy link
Copy Markdown
Member

removing an extra newline too (generator removes it)

@petertseng petertseng merged commit 557c082 into exercism:master Oct 10, 2017
@petertseng
Copy link
Copy Markdown
Member

(I need to change "Removed" to "Remove" - commit messages are imperative)

Thanks for your contribution.

@petertseng petertseng added the sync/readme Keep a README in sync with exercism/problem-specifications label Feb 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sync/readme Keep a README in sync with exercism/problem-specifications

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants