Skip to content

Record develop-staleness and issue-closing learnings in AGENTS.md#103

Merged
ptr727 merged 1 commit into
developfrom
docs/agents-learnings
Jul 3, 2026
Merged

Record develop-staleness and issue-closing learnings in AGENTS.md#103
ptr727 merged 1 commit into
developfrom
docs/agents-learnings

Conversation

@ptr727

@ptr727 ptr727 commented Jun 30, 2026

Copy link
Copy Markdown
Owner

Two durable learnings: mirror main-only fixes to develop (and check git diff origin/develop origin/main before relying on develop), and put Closes #N in the promotion PR. No code changes.

Copilot AI review requested due to automatic review settings June 30, 2026 04:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates AGENTS.md with two durable process learnings for this Docker-only repo: (1) how to handle situations where main receives fixes outside the normal feature -> develop -> main flow, and (2) how to ensure GitHub issue auto-closing triggers on promotion to the default branch.

Changes:

  • Document mirroring main-only fixes back to develop, including a suggested branch-diff check before relying on develop.
  • Document that Closes #N should be placed on the develop -> main promotion PR (or closed manually) so the issue closes on merge to main.

Comment thread AGENTS.md Outdated
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@ptr727 ptr727 merged commit e82de0f into develop Jul 3, 2026
10 checks passed
@ptr727 ptr727 deleted the docs/agents-learnings branch July 3, 2026 18:56
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