Skip to content

Update Pythagorean triplet README.md with missing constraint #668

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

Merged
merged 1 commit into from
Oct 15, 2018

Conversation

TheBestJohn
Copy link
Contributor

Added missing constraint to the Pythagorean triplet problem that is outlined in the linked Project Euler problem.

Added missing constraint to the Pythagorean triplet problem that is outlined in the linked Project Euler problem.
@coriolinus
Copy link
Member

Hi @TheBestJohn, and thank you for contributing this! That does look like a good and useful constraint to add.

Rust gets its problem specifications from the problem-specifications repo, and one of our CI checks ensures that the READMEs for each exercise here correspond with the README there. We therefore can't successfully merge this PR until it is merged upstream, in that repo.

Are you willing to open another PR, making the same change in that repository?

@TheBestJohn
Copy link
Contributor Author

Sure thing. Thanks for the info.

@coriolinus
Copy link
Member

With exercism/problem-specifications#1365 merged, I'm restarting Travis.

@coriolinus coriolinus merged commit 185491d into exercism:master Oct 15, 2018
@petertseng petertseng added the sync/readme Keep a README in sync with exercism/problem-specifications label Nov 18, 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