Skip to content

Fix horse sounds#1443

Closed
kyleplo wants to merge 3 commits intoscratchfoundation:developfrom
kyleplo:develop
Closed

Fix horse sounds#1443
kyleplo wants to merge 3 commits intoscratchfoundation:developfrom
kyleplo:develop

Conversation

@kyleplo
Copy link
Contributor

@kyleplo kyleplo commented Feb 11, 2018

Resolves

#1442

Proposed Changes

Remove "meow" from horse sounds and adds "Horse" and "Horse Gallop"

Reason for Changes

Horses say neigh and they gallop; they don't meow 🐴

Copy link
Contributor

@apple502j apple502j left a comment

Choose a reason for hiding this comment

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

Looks good.

@kyleplo
Copy link
Contributor Author

kyleplo commented Feb 21, 2018

Hang on, I did something wrong. How do you remove commits from a pull request?

@paulkaplan
Copy link
Contributor

As we have mentioned multiple times #1356, please don't submit PRs for issues that are not marked help wanted.

Thanks, but this should be resolved in the source project that JSON is generated from. Please refrain from submitting PRs for issues that are not labeled as “help wanted”.

@paulkaplan paulkaplan closed this Feb 22, 2018
@kyleplo
Copy link
Contributor Author

kyleplo commented Feb 22, 2018

@paulkaplan Sorry.

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

Comments