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.

Sign up for GitHub

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

Jump to bottom

movement update #151

Merged
merged 2 commits into from
Jun 22, 2023
Merged

movement update #151

merged 2 commits into from
Jun 22, 2023

Conversation

grepsedawk
Copy link
Collaborator

@grepsedawk grepsedawk commented Jun 22, 2023 •
edited
Loading

  • Delegate x, y, z to bot.feet
  • Only set movement_action if position is different than current position

Resolves #147

@grepsedawk
Delegate x, y, z to bot.feet
4e64eb3 
Makes for easier position management
@grepsedawk grepsedawk enabled auto-merge (rebase) June 22, 2023 02:14
Gjum
Gjum previously requested changes Jun 22, 2023
View reviewed changes
src/rosegold/control/physics.cr Outdated Show resolved Hide resolved
@grepsedawk grepsedawk requested a review from Gjum June 22, 2023 02:43
Gjum
Gjum requested changes Jun 22, 2023
View reviewed changes
src/rosegold/control/physics.cr Outdated Show resolved Hide resolved
@grepsedawk grepsedawk requested a review from Gjum June 22, 2023 02:49
@grepsedawk grepsedawk merged commit 1ca92cc into main Jun 22, 2023
5 checks passed
@grepsedawk grepsedawk deleted the movement-update branch June 22, 2023 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gjum Gjum Gjum approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bot#move_to fails if player is already in that spot
2 participants
@grepsedawk @Gjum