Skip to content

chore: merge main → canary (resolve README conflict for #73 promote)#74

Merged
joelteply merged 3 commits intocanaryfrom
chore/merge-main-into-canary
Apr 25, 2026
Merged

chore: merge main → canary (resolve README conflict for #73 promote)#74
joelteply merged 3 commits intocanaryfrom
chore/merge-main-into-canary

Conversation

@joelteply
Copy link
Copy Markdown
Contributor

Summary

Resolves the conflict blocking PR #73 (canary → main). Pulls main into canary so the promote-PR can fast-forward cleanly.

Conflict resolution

One file — README.md Core Commands block. The airc join comment had drifted slightly:

Took the canary wording (matches the auto-scope feature shipped in #66 and the skill clarity from #67).

Test plan

  • test/integration.sh — 126/126 still pass
  • No code changes; pure docs conflict

Copilot AI review requested due to automatic review settings April 25, 2026 12:42
@joelteply joelteply merged commit 992dbdd into canary Apr 25, 2026
3 checks passed
@joelteply joelteply deleted the chore/merge-main-into-canary branch April 25, 2026 12:42
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Comment on lines +297 to +300
airc msg "<message>" # broadcast to current room
airc msg @<peer> "<message>" # DM label (still visible to all)
airc send-file <peer> <path> # send a file (scp with airc identity)
airc nick <new-name> # rename your identity; paired peers auto-update
airc nick <new-name> # rename your identity; paired peers auto-update
Copy link

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

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

The Core Commands block now mixes aligned and unaligned spacing (e.g., airc send-file and the Identity commands use different padding before # than the surrounding lines). Consider standardizing formatting across the whole code block (either align all command comments or keep single-spacing) to avoid a visually inconsistent README and noisy future diffs.

Copilot uses AI. Check for mistakes.
Comment thread README.md
airc join <gist-id> # join via shared gist (cross-account fallback)
airc join <mnemonic> # join via humanhash like oregon-uncle-bravo-eleven
airc join # auto-scope to your project's room (or resume prior pairing)
airc join --room <name> # join (or host) a non-general room
Copy link

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

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

airc join --room <name> is described as joining a “non-general room”, but with auto-scope the default room often isn’t #general. Rephrase to something like “join a specific room explicitly” (or explicitly reference #general if that’s what you mean) so the guidance stays accurate.

Suggested change
airc join --room <name> # join (or host) a non-general room
airc join --room <name> # join (or host) a specific room explicitly

Copilot uses AI. Check for mistakes.
Comment thread README.md
Comment on lines +288 to +294
airc join # auto-scope to your project's room (or resume prior pairing)
airc join --room <name> # join (or host) a non-general room
airc join <gist-id> # join via shared gist (cross-account fallback)
airc join <mnemonic> # join via humanhash like oregon-uncle-bravo-eleven

airc list # list open rooms on your gh
airc part # leave current room (host: deletes gist)
airc list # list open rooms on your gh
airc part # leave current room (host: deletes gist)
Copy link

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

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

PR description says this is a one-line wording conflict resolution, but this hunk also reformats multiple lines (spacing/alignment) in the Core Commands code block. If the reformatting isn’t intentional, consider minimizing to the conflict line to keep the merge/promotion diff small.

Copilot uses AI. Check for mistakes.
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.

2 participants