Skip to content

Note that git pr-land publishes its merge in the watch protocol - #160

Merged
cgraf78 merged 1 commit into
mainfrom
watch-protocol-pr-land
Oct 31, 2026
Merged

cgraf78 merged 1 commit into
mainfrom
watch-protocol-pr-land

Conversation

@cgraf78

@cgraf78 cgraf78 commented Oct 31, 2026

Copy link
Copy Markdown
Owner

Summary

The documented gaps say a merge through another tool forgets nothing
until a watcher reports the pull request merged. git-tools'
git pr-land now runs a one-shot git pr-watch right after its merge,
which publishes the merged result, so a landing through it satisfies
the obligation without another watcher run.

  • docs/watch-protocol.md: name git pr-land in that gap and say it
    publishes the merged result itself.
  • Behavior change: none; documentation only.

Testing

  • test/suites/manpage-test: pass (docs wording not pinned).

Merge order

Merge after cgraf78/git-tools#106, which makes git pr-land publish the merged result this note describes.

Summary

The documented gaps say a merge through another tool forgets nothing
until a watcher reports the pull request merged. git-tools'
`git pr-land` now runs a one-shot `git pr-watch` right after its merge,
which publishes the merged result, so a landing through it satisfies
the obligation without another watcher run.

- docs/watch-protocol.md: name `git pr-land` in that gap and say it
  publishes the merged result itself.
- Behavior change: none; documentation only.

Testing

- `test/suites/manpage-test`: pass (docs wording not pinned).
@cgraf78
cgraf78 force-pushed the watch-protocol-pr-land branch from 046d1af to 7cfd184 Compare October 31, 2026 03:12
@cgraf78
cgraf78 merged commit da3dac1 into main Oct 31, 2026
17 checks passed
@cgraf78
cgraf78 deleted the watch-protocol-pr-land branch October 31, 2026 03:19
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