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

[Swift] Fixes forward offset verifiable objects within arrays & fixes failing tests #8135

Merged
merged 2 commits into from
Nov 18, 2023

Conversation

mustiikhalil
Copy link
Collaborator

The following was discovered after a discussion over in the following PR #8134. We didn't create the proper value for the for loop clamping.

@github-actions github-actions bot added the swift label Oct 27, 2023
@mustiikhalil mustiikhalil changed the title Fixes forward offset verifiable objects within arrays Fixes forward offset verifiable objects within arrays & fixes failing tests Oct 27, 2023
@mustiikhalil mustiikhalil self-assigned this Oct 27, 2023
@mustiikhalil mustiikhalil force-pushed the fix-verifier-failing branch 2 times, most recently from 17cb2fa to 07f3b7c Compare October 27, 2023 00:54
@mustiikhalil mustiikhalil changed the title Fixes forward offset verifiable objects within arrays & fixes failing tests [Swift] Fixes forward offset verifiable objects within arrays & fixes failing tests Oct 27, 2023
Fixes failing tests & removes XCTestsManifests
@dbaileychess dbaileychess enabled auto-merge (squash) November 18, 2023 21:13
@dbaileychess dbaileychess merged commit 526c925 into google:master Nov 18, 2023
48 checks passed
@mustiikhalil mustiikhalil deleted the fix-verifier-failing branch November 20, 2023 11:08
candysonya pushed a commit to candysonya/flatbuffers that referenced this pull request Jan 8, 2024
Fixes failing tests & removes XCTestsManifests

Co-authored-by: Derek Bailey <derekbailey@google.com>
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
Fixes failing tests & removes XCTestsManifests

Co-authored-by: Derek Bailey <derekbailey@google.com>
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
Fixes failing tests & removes XCTestsManifests

Co-authored-by: Derek Bailey <derekbailey@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants