Skip to content

chore(claude): approve reading version control and reference docs with testing changes#329

Merged
zimeg merged 2 commits intomainfrom
zimeg-chore-claude-updates
Feb 5, 2026
Merged

chore(claude): approve reading version control and reference docs with testing changes#329
zimeg merged 2 commits intomainfrom
zimeg-chore-claude-updates

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Feb 5, 2026

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

@zimeg zimeg added this to the Next Release milestone Feb 5, 2026
@zimeg zimeg self-assigned this Feb 5, 2026
@zimeg zimeg requested a review from a team as a code owner February 5, 2026 01:05
@zimeg zimeg added code health M-T: Test improvements and anything that improves code health semver:patch Use on pull requests to describe the release version increment labels Feb 5, 2026
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.68%. Comparing base (653e0ed) to head (3412ebf).
⚠️ Report is 1 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@srtaalej srtaalej left a 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 😸

Comment on lines 5 to +29
"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)"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice 🤩

@zimeg
Copy link
Member Author

zimeg commented Feb 5, 2026

@srtaalej I super appreciate the kind review and I'm sure @claude does too 👾 ✨

Please feel free to continue additions to these files! I hear it's best practice to share constant updates of discovered ergonomics. Let's merge this now 🚢 💨

@zimeg zimeg enabled auto-merge (squash) February 5, 2026 19:51
@zimeg zimeg merged commit afbd258 into main Feb 5, 2026
8 checks passed
@zimeg zimeg deleted the zimeg-chore-claude-updates branch February 5, 2026 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code health M-T: Test improvements and anything that improves code health semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants