Skip to content

chore: bump .ai-config to upstream HEAD - #292

Closed
github-actions[bot] wants to merge 3 commits into
mainfrom
automated/bump-submodule
Closed

chore: bump .ai-config to upstream HEAD#292
github-actions[bot] wants to merge 3 commits into
mainfrom
automated/bump-submodule

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated submodule bump of .ai-config.

  • From: f5aed9220b0300e04ffe6a55b3368586a7bf1fcd
  • To: 0e4eed6215d160bf27e074eaf5719b39ea1d5d0e

Generated by the bump-submodule reusable workflow in
d-morrison/gha.

@codecov

codecov Bot commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@github-actions
github-actions Bot force-pushed the automated/bump-submodule branch from 8f1adcd to 87bb470 Compare August 24, 2026 06:08
@d-morrison
d-morrison force-pushed the automated/bump-submodule branch from 87bb470 to a8a2f81 Compare August 31, 2026 06:02
The bot's original pin had fallen behind upstream `main`; refresh it to the
current HEAD so the PR delivers what its title claims.

Merge `main` to clear the stale base, bump the dev version so version-check
passes, and add the NEWS.md entry the changelog check requires.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017trXKZFoCYC4kP7NKNnN6F

Copy link
Copy Markdown
Member

/review

Pushed three fixes to this bot PR, which was failing two checks and had gone stale:

  • Refreshed the submodule pin. The bot's original pin (f5aed92f68dd34) had itself fallen behind upstream; .ai-config now points at eeeb6ac, the current Morrison-Lab/ai-config main, so the PR delivers what its title claims.
  • Merged main. The base had drifted from 3b47161 to 5be9001.
  • Bumped the dev version to 0.1.0.9019 and added a NEWS.md entry, which fixes the failing version-check and Check Changelog Action.

Heads-up on merge order: this PR, #284, #298, #310 and #311 all carry DESCRIPTION version 0.1.0.9019, so whichever merges second will conflict there and need a re-bump. Tracked as #312.


Generated by Claude Code

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

👀 Claude Review Dispatchedrun reviewing PR #292 at c433bc9.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Claude finished reviewView run


This is a minimal, mechanical PR: bump the .ai-config submodule pin, bump the dev version (0.1.0.90180.1.0.9019), and add a matching NEWS.md entry. This follows the exact pattern of the prior entries in the "Internal" section (e.g., the submodule-bump workflow this repo already uses per CLAUDE.md). I independently verified the new submodule SHA (eeeb6ac6ef7059e967f1fcea72365b6ebcb0f56d) resolves to a real, legitimate commit on Morrison-Lab/ai-config main (fetched it directly) — not a hallucinated or dangling reference. Version and NEWS.md bumps are consistent and match the format required by this repo's version-check / changelog-check CI gates, as referenced in the PR's own description.

No CLAUDE.md violations apply here — there's no R/Quarto code in this diff (only DESCRIPTION, NEWS.md, and a submodule pointer), so the lab-manual coding-style/practices rules and the idiomatic-code review focus items in the repo's CLAUDE.md don't have anything to check against. No hallucinated APIs, functions, or citations. The one thing flagged by the PR author (d-morrison) themselves — a version-number collision with sibling PRs #284/#298/#310/#311, all also bumping to 0.1.0.9019 — is already tracked separately in #312 and isn't something to block this PR on.

No prior Claude review comment with a ### Verdict line exists on this PR, so this is a first-pass review, not a confirming one.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.0",
  "reviewer": "claude",
  "commit_sha": "c433bc97cbfa24d0b6c4fb72007b283fa21bda7b",
  "verdict": "CLEAN",
  "findings": []
}

Reviewed commit: c433bc9

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

💰 Cost: $0.2381 (review) — run

Copy link
Copy Markdown
Member

For the record: I pushed 2f8b19f (merge of main) and c433bc9 (refreshed submodule pin, version bump, NEWS entry) to this bot-opened branch. Per @d-morrison, I should only be working PRs they opened or are assigned to, and this one is neither. The commits stand; I have stopped watching this PR.


Generated by Claude Code

Copy link
Copy Markdown
Member

Retracting the stand-down above: per @d-morrison, workflow-opened PRs like this one are in scope, so this PR is back under watch.

State is unchanged since the review: head c433bc9, every check green, and the review on that exact commit reads Ready for merge with no findings. Nothing further to push; it awaits a human merge. Note the DESCRIPTION version collision with #284, #298, #310, #311 and #313 (all at 0.1.0.9019, tracked in #312): whichever merges second needs a re-bump.


Generated by Claude Code

Copy link
Copy Markdown
Member

Closing: #317 merged as 1118ac7 and removed the .ai-config submodule, .gitmodules, and the bump-submodule workflow, so this bump has nothing left to update. The corpus now reaches Claude Code sessions and the CI reviewer as the ai-config@Morrison-Lab plugin instead.

Posted by Claude Code (AI agent) --- not written by a human.


Generated by Claude Code

@d-morrison d-morrison closed this Sep 2, 2026
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