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

[deps] Update recastnavigation commit #6608

Open
wants to merge 1 commit into
base: base
Choose a base branch
from

Conversation

WinterSolstice8
Copy link
Member

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

No change should be observed, but some warnings should be gone on build, and undefined behavior should be gone.

Steps to test these changes

Notice nothing different in game
Wait for CI, check for some CI build warnings

No change should be observed, but some warnings should be gone on build, and undefined behavior should be gone.
@WinterSolstice8
Copy link
Member Author

Strange. This doesnt seem to fix the CI build issue. Check ClangTidy 64 bit and it still reports the errors...

@WinterSolstice8 WinterSolstice8 added the hold On hold, pending further action/info label Dec 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold On hold, pending further action/info
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants