Skip to content

Reduce CI usage#8256

Open
link2xt wants to merge 1 commit into
mainfrom
link2xt/reduce-ci-usage
Open

Reduce CI usage#8256
link2xt wants to merge 1 commit into
mainfrom
link2xt/reduce-ci-usage

Conversation

@link2xt

@link2xt link2xt commented May 20, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

macOS runners are slow
and for PyPy users should switch to JSON-RPC bindings.
We are only running legacy Python tests
to test the Rust code behaivor and CFFI,
but not the Python bindings,
so testing one implementation is enough.
@link2xt link2xt force-pushed the link2xt/reduce-ci-usage branch 2 times, most recently from 8ec4151 to 19d3485 Compare May 20, 2026 02:57
@Hocuri

Hocuri commented May 20, 2026

Copy link
Copy Markdown
Collaborator

This means we won't test the CFFI on Macos anymore? Isn't this a problem because Deltachat-iOS uses the CFFI? Or is it fine because we don't change it, so we assume that it will keep working?

@Simon-Laux

Simon-Laux commented Jun 28, 2026

Copy link
Copy Markdown
Member

Or is it fine because we don't change it, so we assume that it will keep working?

I doubt that it is likely that CFFI suddenly fails on an old method.
Also, many tests for features were moved to jsonrpc, and CFFI is only a thin wrapper that is not fully tested anyway AFAIK.

I don't think its likely that our thin wrapper crate contains something in the older methods that suddenly breaks in newer operating system versions and in newly added methods we would see it during testing if something doesn't work only on macOS.

As long as deltachat's C interface keeps not doing platform specific stuff (like not using Objective-C types), I'd say it's fine to just test it on Linux.

An alternative could be to only enable macOS CFFI testing on release tags, that would be enough to catch such an unlikely issue while still saving CI time.

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.

3 participants