Repository navigation
fix: guidelines name the T in the date command - #9
Merged
Merged
Conversation
assistant-guidelines.md requires prompt files named YYYY-MM-DDTHH-MM-SS-XXX_<model>.txt and stresses that the T matters, but told the model to run `date +%Y-%m-%d%H-%M-%S`, which leaves it out. It now says `date +%Y-%m-%dT%H-%M-%S`, the format both hooks write with strftime. A test pins the documented format, in the .github copy as well as the bundled one, since ./release copies the former over the latter. Prompt: (comment from the review process) fix in [ai-prompt-auto-commit] and make a PR [on a review finding that the guidelines' date command lacks the T its filename pattern demands] Followup-Prompt: [fix review findings] Assisted-By: Claude Opus 5.5 Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
`git add .` after `pre-commit autoupdate` stages every unrelated change and untracked file in the tree. All pre-commit needs is the updated .pre-commit-config.yaml, which it refuses to run with unstaged; say that, and stage just that file. Prompt: [fix review findings] Assisted-By: Claude Opus 5.5 Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
Member
|
Thanks for the improvements. I have tried them out and we can upgrade to version 0.10. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Disclaimer: the code and the text below the line are AI-generated, by Claude Opus 5.5. The bug was caught by a clean-context AI-review of caldav.
assistant-guidelines.mdrequires prompt files namedYYYY-MM-DDTHH-MM-SS-XXX_<model>.txtand stresses that theTmatters, but told the model to rundate +%Y-%m-%d%H-%M-%S, which leaves it out. It now saysdate +%Y-%m-%dT%H-%M-%S, the format both hooks write withstrftime. A test pins the documented format in both copies of the guidelines, since./releasecopies the.githubone over the bundled one.A second commit makes the README's update recipe stage only
.pre-commit-config.yamlinstead ofgit add ., which also staged every unrelated change in the tree.🤖 Generated with Claude Code