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

Checking abi #710

Closed
wants to merge 11 commits into from
Closed

Checking abi #710

wants to merge 11 commits into from

Conversation

Y-Less
Copy link
Collaborator

@Y-Less Y-Less commented Sep 2, 2023

I've merged a few things, so this isn't a true PR, just wanted to see the ABI tool output before pushing to master and closing the PRs.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 4 committers have signed the CLA.

✅ Abyss-z94
✅ Hual
✅ WoutProvost
❌ Alex Cole


Alex Cole seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Y-Less
Copy link
Collaborator Author

Y-Less commented Sep 2, 2023

Yes I thought that was the case. It's the PawnLookup struct, which isn't used in the public API, but is in the generic includes because it can be used by both the pawn component and plugins internally. Thus, can be ignored.

@Y-Less Y-Less closed this Sep 2, 2023
@AmyrAhmady AmyrAhmady deleted the checking-abi branch September 22, 2023 06:21
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.

5 participants