Skip to content

Disabling Claude attribution#7146

Merged
theMickster merged 1 commit intomainfrom
disabling-claude-attribution
Mar 4, 2026
Merged

Disabling Claude attribution#7146
theMickster merged 1 commit intomainfrom
disabling-claude-attribution

Conversation

@theMickster
Copy link
Contributor

🎟️ Tracking

📔 Objective

Disabling Claude attribution

@theMickster theMickster added the ai-review Request a Claude code review label Mar 4, 2026
@theMickster theMickster requested a review from a team as a code owner March 4, 2026 20:55
@theMickster theMickster enabled auto-merge (squash) March 4, 2026 20:55
@claude
Copy link
Contributor

claude bot commented Mar 4, 2026

Claude finished @theMickster's task in 52s —— View job


Pull Request Review: Disabling Claude Attribution

Reviewed commit: 4d67e8d

Change Summary: Adds an attribution block to .claude/settings.json with empty strings for commit and pr, which disables Claude Code attribution text on commits and pull requests.

Risk Assessment: Very Low

Findings: None

This is a clean, minimal configuration change limited to developer tooling. The JSON is valid, the change is well-scoped to a single file (.claude/settings.json), and there is no impact on application code, security, tests, or production behavior.

No issues identified. Approved.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 4, 2026

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

Logo
Checkmarx One – Scan Summary & Details306fc41d-69e4-4616-99dc-b702bf269e8a

Great job! No new security vulnerabilities introduced in this pull request

@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.68%. Comparing base (2b663d7) to head (4d67e8d).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7146   +/-   ##
=======================================
  Coverage   56.68%   56.68%           
=======================================
  Files        2025     2025           
  Lines       88640    88640           
  Branches     7905     7905           
=======================================
+ Hits        50242    50243    +1     
+ Misses      36572    36571    -1     
  Partials     1826     1826           

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@theMickster theMickster merged commit 2e8807d into main Mar 4, 2026
44 checks passed
@theMickster theMickster deleted the disabling-claude-attribution branch March 4, 2026 21:30
prograhamming pushed a commit that referenced this pull request Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants