Skip to content

Add OPERATIONS.md and clang-format standards to the backlog - #443

Merged
ptr727 merged 3 commits into
developfrom
backlog-operations-clangformat
Jul 25, 2026
Merged

Add OPERATIONS.md and clang-format standards to the backlog#443
ptr727 merged 3 commits into
developfrom
backlog-operations-clangformat

Conversation

@ptr727

Copy link
Copy Markdown
Owner

Two backlog items from the HA/ESPHome convergence: (1) standardize OPERATIONS.md as the fleet's extraction target for operational content pulled out of a carried AGENTS.md (both repos' agents converged on it); (2) add a canonical clang-format config for the cpp type - the C++ analogue of the shared ruff config - so cpp.clangformat.shared references one style, based on the ESPHome agent's proposal.

Docs-only. spec/validate.py OK.

🤖 Generated with Claude Code

Two fleet-standard opportunities surfaced by the config-repo convergence:
a standard OPERATIONS.md as the extraction target for operational content
pulled out of a carried AGENTS.md, and a canonical clang-format config for
the cpp type (the C++ analogue of the shared ruff config).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
CopilotAI review requested due to automatic review settings July 25, 2026 17:36

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds two new backlog items to the README's TODO list to capture fleet-standardization work identified during Home Assistant / ESPHome convergence, keeping the guidance in a committed location.

Changes:

  • Add a backlog item to standardize OPERATIONS.md as the extraction target for repo-specific operational/runbook content.
  • Add a backlog item to define a fleet-standard .clang-format style for the cpp type (catalog snippet + CODESTYLE C++ section) to anchor cpp.clangformat.shared.

cspell checks .md files and does not know the identifier clangformat (from
the cpp.clangformat.shared check id, which lives in a .json cspell does not
check). Refer to the cpp clang-format check descriptively instead.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
CopilotAI review requested due to automatic review settings July 25, 2026 17:41

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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 1 comment.

Comment threadREADME.md Outdated
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
CopilotAI review requested due to automatic review settings July 25, 2026 17:47

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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 b719d3b into developJul 25, 2026
7 checks passed
@ptr727
ptr727 deleted the backlog-operations-clangformat branch July 25, 2026 17:53
ptr727 added a commit that referenced this pull request Jul 25, 2026
#446)
## Promote develop to main
Three doc/spec-only changes (no code path, no release):
- **#443** - backlog: standardize `OPERATIONS.md` as the extraction
target for operational content, and add a canonical clang-format config
for the `cpp` type.
- **#444** - name the post-push review race in the merge gate (AGENTS.md
+ copilot-instructions): a push turns checks green before the re-review
lands, so confirm a review on the current head SHA before reading
merge-state.
- **#445** - bridge the CODESTYLE Python profiles (Project/Scripts) to
the declared `build`/`lint-only` names.
## Verification
`spec/validate.py` OK; `spec/audit.py --selftest` PASS.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign up for freeto 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

@ptr727