Skip to content

Link a pre-cruft repo at the commit it last matched - #23

Merged
Marius1311 merged 2 commits into
mainfrom
docs/cruft-link-commit
Oct 16, 2026
Merged

Marius1311 merged 2 commits into
mainfrom
docs/cruft-link-commit

Conversation

@Marius1311

Copy link
Copy Markdown
Member

cruft link links at the template's latest commit unless another is entered at its prompt, and cruft update then finds nothing to apply, so a pre-cruft repo never receives the changes it missed. The README now says to enter the commit the repo last matched.

Tested: a project rendered at d58142f, linked there, then cruft update brought in every later change (PR template, review guide, AGENTS.md, workflows).

🤖 Generated with Claude Code

https://claude.ai/code/session_011aQpF4MA9v7dTVK1FqMMVJ

Linking at the default, the latest commit, leaves cruft update nothing to apply.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011aQpF4MA9v7dTVK1FqMMVJ
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-15T16:11:52.932844Z f7f39ee PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f7f39eed1c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md Outdated
`uvx cruft link https://github.com/quadbio/analysis_template`.
A repo created before this template used cruft is linked once with
`uvx cruft link https://github.com/quadbio/analysis_template`. At its commit prompt, enter the template commit
whose files the repo last matched, not the default latest; `cruft update` then applies everything since.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use a renderable baseline for pre-cruft repositories

A genuinely pre-cruft project last matched commit 2419048 or earlier, but those commits contain neither cookiecutter.json nor the templated project directory; both first appear in 88ea222. Following this instruction therefore records a non-template SHA, and the next update fails because Cruft generates its old baseline by checking out and rendering the recorded commit. Document a renderable migration baseline or a separate conversion procedure rather than directing these repositories to their actual last-matching commit.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed in bc2915e: link no earlier than 88ea222, the first commit cruft can render.

cruft update renders the linked commit; commits before 88ea222 have no cookiecutter.json.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011aQpF4MA9v7dTVK1FqMMVJ
@Marius1311
Marius1311 merged commit ed9c5e9 into main Oct 16, 2026
1 check passed
@Marius1311
Marius1311 deleted the docs/cruft-link-commit branch October 16, 2026 10:03
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