Skills for vibe coding#2201
Conversation
TaranDahl
added
Tested
⚙️T1
|
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
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
|
|
||
| **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.
TaranDahl
added
the
Will be merged in 24h
5 participants
Added the skills of Write Documentation, Generate INI tag and Check Hooks.