-
Notifications
You must be signed in to change notification settings - Fork 26
chore(claude): approve reading version control and reference docs with testing changes #329
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
Conversation
…h testing changes
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #329 +/- ##
==========================================
+ Coverage 64.67% 64.68% +0.01%
==========================================
Files 212 212
Lines 17778 17778
==========================================
+ Hits 11498 11500 +2
Misses 5205 5205
+ Partials 1075 1073 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
srtaalej
left a comment
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.
thank you for these changes to our .claude folder! cant wait to try it out 😸
| "Bash(gofmt:*)", | ||
| "Bash(gh issue view:*)", | ||
| "Bash(gh label list:*)", | ||
| "Bash(gh pr checks:*)", | ||
| "Bash(gh pr diff:*)", | ||
| "Bash(gh pr list:*)", | ||
| "Bash(gh pr status:*)", | ||
| "Bash(gh pr update-branch:*)", | ||
| "Bash(gh pr view:*)", | ||
| "Bash(gh search code:*)", | ||
| "Bash(git diff:*)", | ||
| "Bash(git grep:*)", | ||
| "Bash(git log:*)", | ||
| "Bash(git status:*)", | ||
| "Bash(go mod graph:*)", | ||
| "Bash(go mod tidy:*)", | ||
| "Bash(grep:*)", | ||
| "Bash(ls:*)", | ||
| "Bash(make build:*)", | ||
| "Bash(make build-ci:*)", | ||
| "Bash(make lint:*)" | ||
| "Bash(make lint:*)", | ||
| "Bash(make test:*)", | ||
| "Bash(tree:*)", | ||
| "WebFetch(domain:github.com)", | ||
| "WebFetch(domain:docs.slack.dev)" |
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.
nice 🤩
Summary
This PR follows #324 with an update to testing recommendations and also adds version control tools to the approved list of commands for Claude 👾
Requirements