Skip to content

docs: document L2CAP support and add a build workflow - #15

Closed
acouvreur wants to merge 2 commits into
fix/l2cap-stream-eventsfrom
docs/l2cap-readme-ci
Closed

acouvreur wants to merge 2 commits into
fix/l2cap-stream-eventsfrom
docs/l2cap-readme-ci

Conversation

@acouvreur

Copy link
Copy Markdown
Member

Stacked on #14.

L2CAP support went in without a mention in the README, so the only way to learn that channels exist, and that reads and writes on them do not block, is to read the source.

The repository also has no CI at all. Nothing compiles a proposed change before it is merged, on a codebase where most of the surface is Objective C reached through cgo and can only be built on macOS. A mistake there is invisible to review and only surfaces once someone pulls the change into their own project.

A macOS build and vet job puts a compiler in front of changes before they land, and a short README section makes the feature discoverable.

Note that the workflow only starts running once this is on master, since GitHub takes the workflow definition for a pull request from its base branch.

@acouvreur
acouvreur force-pushed the docs/l2cap-readme-ci branch from 7e6ae35 to 77b30bc Compare August 26, 2026 03:10
@acouvreur
acouvreur force-pushed the docs/l2cap-readme-ci branch from 77b30bc to 0420ab0 Compare August 26, 2026 03:11
@acouvreur

Copy link
Copy Markdown
Member Author

Reopening with the vet fixes as its base so the workflow this adds is green on the first run rather than red the moment it reaches master.

@acouvreur acouvreur closed this Aug 26, 2026
@deadprogram
deadprogram deleted the docs/l2cap-readme-ci branch August 27, 2026 06:10
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