Skip to content

Remove stubs entirely#8983

Open
abadams wants to merge 1 commit intomainfrom
abadams/remove_stubs
Open

Remove stubs entirely#8983
abadams wants to merge 1 commit intomainfrom
abadams/remove_stubs

Conversation

@abadams
Copy link
Member

@abadams abadams commented Mar 4, 2026

stubs never really moved beyond proof-of-concept. E.g. there is no cmake support for them, and the one app that uses them has no build.

This PR removes them entirely. Opened for discussion.

stubs never really moved beyond proof-of-concept. E.g. there is no cmake
support for them, and the one app that uses them has no build.

This PR removes them entirely. Opened for discussion.

Co-authored-by: Claude Code <noreply@anthropic.com>
@alexreinking alexreinking self-requested a review March 4, 2026 23:36
Copy link
Member

@alexreinking alexreinking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In principle, I agree with this change. It removes a source of extreme complexity with no users. Almost 2000 lines removed is a remarkable metric. We should probably send a notice to the mailing list, in case anyone there is reading and cares to weigh in.

@alexreinking alexreinking added the release_notes For changes that may warrant a note in README for official releases. label Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_notes For changes that may warrant a note in README for official releases.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants