Skip to content

chore: settings adjustment#1959

Merged
elibosley merged 1 commit intomainfrom
codex/disable-claude-coauthor
Mar 24, 2026
Merged

chore: settings adjustment#1959
elibosley merged 1 commit intomainfrom
codex/disable-claude-coauthor

Conversation

@Ajit-Mehrotra
Copy link
Contributor

@Ajit-Mehrotra Ajit-Mehrotra commented Mar 24, 2026

Summary

Small settings adjustment.

Testing

Not run (config-only change).

Summary by CodeRabbit

  • Chores
    • Updated configuration settings file with additional metadata fields and formatting improvements.

- Purpose: make a small configuration adjustment for Claude project settings.
- Before: Claude's default attribution settings were left implicit for this repo.
- Why: that behavior was adding commit and PR attribution that we do not want here.
- What: explicitly set commit and PR attribution to empty strings in the shared Claude settings.
- How: update .claude/settings.json with the supported attribution keys while preserving the existing permissions block.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 4dd5f799-135d-4e31-9632-f5dad8e11874

📥 Commits

Reviewing files that changed from the base of the PR and between ea41225 and 05d7154.

📒 Files selected for processing (1)
  • .claude/settings.json

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


Walkthrough

A new attribution configuration object with commit and pr fields has been added to the Claude settings JSON file. The file formatting has also been corrected to include a trailing newline.

Changes

Cohort / File(s) Summary
Configuration Update
.claude/settings.json
Added new attribution object with commit and pr fields (both empty strings); corrected file formatting with trailing newline.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A settings file grows with care,
New attribution fields appear,
With commit and pr standing tall,
Configuration answered the call,
Now proper newlines close the day!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/disable-claude-coauthor

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@elibosley elibosley merged commit 26d5f8b into main Mar 24, 2026
6 of 7 checks passed
@elibosley elibosley deleted the codex/disable-claude-coauthor branch March 24, 2026 19:41
@codecov
Copy link

codecov bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.95%. Comparing base (ea41225) to head (05d7154).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1959   +/-   ##
=======================================
  Coverage   51.94%   51.95%           
=======================================
  Files        1030     1030           
  Lines       71121    71121           
  Branches     7933     7937    +4     
=======================================
+ Hits        36947    36951    +4     
+ Misses      34051    34047    -4     
  Partials      123      123           

☔ 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.

@github-actions
Copy link
Contributor

This plugin has been deployed to Cloudflare R2 and is available for testing.
Download it at this URL:

https://preview.dl.unraid.net/unraid-api/tag/PR1959/dynamix.unraid.net.plg

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants