Skills for vibe coding#2201
Conversation
|
Nightly build for this pull request:
This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build. |
|
To Chinese users:
|
Metadorius
left a comment
There was a problem hiding this comment.
partial review, skills only
also we don't use RIP, EIP is the one we use
|
|
||
| After user approval, write the approved text into the appropriate doc file. For sections in `docs/New-or-Enhanced-Logics.md` or similar, insert the new `### ` section in alphabetical order among existing sibling headings under the correct `## ` category. Look at existing sibling `### ` headings to determine the correct insertion point. | ||
|
|
||
| Continue with the INI code block following existing conventions: |
|
|
||
| **Mode A — Specify a commit** (user provides a SHA or commit name): | ||
| ``` | ||
| python discover_hooks.py --commit <sha> |
There was a problem hiding this comment.
is this needed? IMO it should be simpler, just use git instead
There was a problem hiding this comment.
remove the commit stuff from the script and just instruct the bot to checkout the commit hash
There was a problem hiding this comment.
I believe any problem that can be solved with a script should best be handled by a script rather than an LLM. LLMs are not entirely reliable; different models vary greatly in capability (not everyone can afford Opus, myself included 🤣), and their performance fluctuates over time and by probability. In contrast, a script always produces consistent results.
There was a problem hiding this comment.
you can make a .bat to have git checkout and then python discover_hooks.py
There was a problem hiding this comment.
besides, for LLM, what's the difference between
python discover_hooks.py --commit <sha>and
git checkout <sha> && python discover_hooks?
There was a problem hiding this comment.
you can make a .bat to have git checkout and then python discover_hooks.py
Checkout could be interrupted by things like stash conflict, which is hard to handle by LLM. Also, you must commit or stash everything before checkout, which is inconvenient.
Added the skills of Write Documentation and Generate INI tag