mathieu/murmur
Watch
1
1
Fork
You've already forked murmur
 0
Code Issues 35 Pull requests 11 Projects Releases Packages Wiki Activity Actions

fix(ipc): préserver les événements bufferisés après Subscribe #36

Open
hermes wants to merge 0 commits from fix/ipc-subscribe-decoder into main
pull from: fix/ipc-subscribe-decoder
merge into: mathieu:main
Conversation 0 Commits - Files changed -
hermes commented 2026-08-07 17:04:03 +00:00
First-time contributor
Copy link

Résumé

  • réutilise le même json.Decoder pour l’acquittement Subscribe et le flux d’événements
  • ajoute une régression où acquittement et premier événement arrivent dans le même Write
  • laisse le framing one-shot de Do inchangé

Closes #11

Validation

go test -race -count=100 ./internal/ipc

Résultat: OK (100 répétitions sous race).

## Résumé - réutilise le même `json.Decoder` pour l’acquittement Subscribe et le flux d’événements - ajoute une régression où acquittement et premier événement arrivent dans le même `Write` - laisse le framing one-shot de `Do` inchangé Closes #11 ## Validation ```text go test -race -count=100 ./internal/ipc ``` Résultat: OK (100 répétitions sous race).
fix(ipc): preserve buffered subscription events
Some checks failed
CI / amd64 / Go 1.23.12 (push) Has been cancelled
Details
CI / amd64 / Go 1.26.5 (push) Has been cancelled
Details
CI / arm64 cross-build / Go 1.23.12 (push) Has been cancelled
Details
CI / arm64 cross-build / Go 1.26.5 (push) Has been cancelled
Details
CI / amd64 / Go 1.23.12 (pull_request) Has been cancelled
Details
CI / amd64 / Go 1.26.5 (pull_request) Has been cancelled
Details
CI / arm64 cross-build / Go 1.23.12 (pull_request) Has been cancelled
Details
CI / arm64 cross-build / Go 1.26.5 (pull_request) Has been cancelled
Details
6c646da241
Some checks failed
CI / amd64 / Go 1.23.12 (push) Has been cancelled
Details
CI / amd64 / Go 1.26.5 (push) Has been cancelled
Details
CI / arm64 cross-build / Go 1.23.12 (push) Has been cancelled
Details
CI / arm64 cross-build / Go 1.26.5 (push) Has been cancelled
Details
CI / amd64 / Go 1.23.12 (pull_request) Has been cancelled
Details
CI / amd64 / Go 1.26.5 (pull_request) Has been cancelled
Details
CI / arm64 cross-build / Go 1.23.12 (pull_request) Has been cancelled
Details
CI / arm64 cross-build / Go 1.26.5 (pull_request) Has been cancelled
Details
This branch is already included in the target branch. There is nothing to merge.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/ipc-subscribe-decoder:fix/ipc-subscribe-decoder
git switch fix/ipc-subscribe-decoder

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff fix/ipc-subscribe-decoder
git switch fix/ipc-subscribe-decoder
git rebase main
git switch main
git merge --ff-only fix/ipc-subscribe-decoder
git switch fix/ipc-subscribe-decoder
git rebase main
git switch main
git merge --no-ff fix/ipc-subscribe-decoder
git switch main
git merge --squash fix/ipc-subscribe-decoder
git switch main
git merge --ff-only fix/ipc-subscribe-decoder
git switch main
git merge fix/ipc-subscribe-decoder
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
mathieu/murmur!36
No description provided.