-
Notifications
You must be signed in to change notification settings - Fork 300
Start on use_claude_code()
#2203
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
inst/claude/settings.json
Outdated
| "Bash(Rscript:*)", | ||
| "Bash(rm:*)", | ||
| "Bash(air:*)", | ||
| "WebFetch(domain:github.com)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| "WebFetch(domain:github.com)" | |
| "WebFetch(domain:github.com)", | |
| "WebFetch(domain:raw.githubusercontent.com)" |
I also end up with this one in most projects.
Other candidates I've accrued in local settings recently that might be generally useful (Claude 🫶 the gh cli):
...
"Bash(cat:*)",
"Bash(git grep:*)",
"Bash(gh pr view:*)",
"Bash(gh issue view:*)",
"Bash(gh api:*)",
"Bash(gh pr diff:*)",
"Bash(gh issue list:*)",
"WebFetch(domain:cran.r-project.org)",
...
We should probably regard inst/claude/settings.json as the main source of truth, then that will propagate to usethis's .claude/settings.json by applying use_claude_code() to itself.
|
Need something for news — i.e. alphbaetise by function name, and should be single line. Alphabetically order. Do we need to explicitly allow skills? |
Co-authored-by: Jennifer (Jenny) Bryan <[email protected]>
Fixes #2195
Note that something seems wrong with the permissions — claude still asks me if it's ok to edit various files and run
Rscript. But it doesgrepandlswithout confirmation.