Skip to content

Automatically check that safe and native signatures are in-sync - #761

Draft
shish wants to merge 1 commit into
masterfrom
push-xzrrplkxnosn
Draft

shish wants to merge 1 commit into
masterfrom
push-xzrrplkxnosn

Conversation

@shish

@shish shish commented Oct 24, 2026

Copy link
Copy Markdown
Collaborator

I fear that this is going to have a lot of false-positives (eg cases
where phpstan's custom types library adds extra context which doesn't
exist natively), but let's try it and see...

@shish
shish marked this pull request as draft October 24, 2026 12:23
@shish
shish force-pushed the push-xzrrplkxnosn branch from ffdba3e to eec1d11 Compare October 24, 2026 12:45
I fear that this is going to have a lot of false-positives (eg cases
where phpstan's custom types library adds extra context which doesn't
exist natively), but let's try it and see...
@shish
shish force-pushed the push-xzrrplkxnosn branch from eec1d11 to ef1109e Compare October 24, 2026 14:06
@shish

shish commented Oct 24, 2026

Copy link
Copy Markdown
Collaborator Author

Yup, that sure is a lot of false-positives :( I like the idea, but it's going to need quite a bit of work to get the multiple sources of truth in-sync...

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.

1 participant