Skip to content

[ffigen] More test regen infra#3317

Merged
liamappelbe merged 5 commits into
mainfrom
ffigen_test_infra_3
Apr 21, 2026
Merged

[ffigen] More test regen infra#3317
liamappelbe merged 5 commits into
mainfrom
ffigen_test_infra_3

Conversation

@liamappelbe
Copy link
Copy Markdown
Contributor

@liamappelbe liamappelbe commented Apr 20, 2026

  • Add an integration test verifying that the libclang bindings are up to date
  • Migrate the JSON schema test to use the same file matching infra as everything else
  • Delete generate_json_schema.dart, since it's now redundant with the UPDATE=true dart test workflow

All generated code in the FFIgen package is now covered by the same testing and updating infra.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 20, 2026

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

This check can be disabled by tagging the PR with skip-changelog-check.

Breaking changes ✔️
Package Change Current Version New Version Needed Version Looking good?

This check can be disabled by tagging the PR with skip-breaking-check.

API leaks ✔️

The following packages contain symbols visible in the public API, but not exported by the library. Export these symbols or remove them from your publicly visible API.

Package Leaked API symbol Leaking sources

This check can be disabled by tagging the PR with skip-leaking-check.

@liamappelbe liamappelbe changed the title WIP [ffigen] More test infra WIP [ffigen] More test regen infra Apr 20, 2026
@liamappelbe liamappelbe changed the title WIP [ffigen] More test regen infra [ffigen] More test regen infra Apr 20, 2026
@liamappelbe liamappelbe marked this pull request as ready for review April 20, 2026 04:33
@liamappelbe liamappelbe requested a review from dcharkes April 20, 2026 04:33
@liamappelbe liamappelbe merged commit 2052162 into main Apr 21, 2026
44 of 50 checks passed
@liamappelbe liamappelbe deleted the ffigen_test_infra_3 branch April 21, 2026 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants