Skip to content
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

Print "You pushed your mount" only if it was really done #55242

Merged
merged 1 commit into from
Feb 11, 2022
Merged

Print "You pushed your mount" only if it was really done #55242

merged 1 commit into from
Feb 11, 2022

Conversation

Night-Pryanik
Copy link
Contributor

Summary

Interface "Print 'You pushed your mount' only if it was really done."

Purpose of change

Previously if you tried to push your mount (e.g. a horse) into a terrain with SHARP flag (e.g. a rose bush), the log will tell you that You pushed your mount, despite the horse stood in place.

Describe the solution

Check if your mount will move to the location you pushed it to before printing the message.

Describe alternatives you've considered

None.

Testing

Placed my horse next to the rose bush. Look at my horse, my horse is amazing!
изображение
Tried to push it into the bush. My horse stood in place, the log correctly said it resisted to move.
Tried to push it to the empty place. The horse moved to that place, the log correctly said that I pushed my horse.

Additional context

None.

@Night-Pryanik Night-Pryanik added [C++] Changes (can be) made in C++. Previously named `Code` Monsters Monsters both friendly and unfriendly. labels Feb 9, 2022
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Feb 9, 2022
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 9, 2022
@kevingranade kevingranade merged commit cd8bacd into CleverRaven:master Feb 11, 2022
@Night-Pryanik Night-Pryanik deleted the clarify-messages-about-pushing-mounts branch February 12, 2022 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions Monsters Monsters both friendly and unfriendly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants