Skip to content

Update code of conduct to require ai notices on communication - #9453

Merged
abadams merged 2 commits into
mainfrom
abadams/agent_github_messages
Sep 16, 2026
Merged

abadams merged 2 commits into
mainfrom
abadams/agent_github_messages

Conversation

@abadams

@abadams abadams commented Sep 15, 2026

Copy link
Copy Markdown
Member

Claude posted a comment as me autonomously with no indication that it was bot-generated. I don't like this at all. This PR changes the code of conduct and CLAUDE.md in an effort to get it to not do this in future. You should know whether you're talking to me or an agent acting under my username.

@abadams

abadams commented Sep 15, 2026

Copy link
Copy Markdown
Member Author

It seems pretty innocuous, but because it's a change to the code of conduct I'd like sign-off from some more people. I think as a general policy it should require broad consensus to change the code of conduct. I'll tag it dev-meeting.

@abadams abadams added the dev_meeting Topic to be discussed at the next dev meeting label Sep 15, 2026
@mcourteaux mcourteaux added the skip_buildbots Do not run buildbots on this PR. Must add before opening PR as we scan labels immediately. label Sep 15, 2026
@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.10%. Comparing base (4c5525c) to head (562752e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9453      +/-   ##
==========================================
- Coverage   70.12%   70.10%   -0.02%     
==========================================
  Files         261      261              
  Lines       79818    79818              
  Branches    19445    19445              
==========================================
- Hits        55972    55957      -15     
+ Misses      17991    17973      -18     
- Partials     5855     5888      +33     

☔ View full report in Codecov by Harness.
📢 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.

@mcourteaux

Copy link
Copy Markdown
Contributor

I don't really think we need many opinions here. We humans are telling robots how to behave and that's fine.

@alexreinking

Copy link
Copy Markdown
Member

I don't really think we need many opinions here. We humans are telling robots how to behave and that's fine.

I agree. Not much to discuss. I'll tag @derek-gerstmann and @slomp in case they have any objections.

@slomp

slomp commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

No objections.

@abadams

abadams commented Sep 16, 2026

Copy link
Copy Markdown
Member Author

Yes, I didn't imagine this change would be controversial, I just don't like the idea that one person and one approver can change the code of conduct without others having the chance to weigh in.

@derek-gerstmann

Copy link
Copy Markdown
Contributor

No controversy for me ... I approve.

@abadams

abadams commented Sep 16, 2026

Copy link
Copy Markdown
Member Author

I think that's enough sign-off. I'll merge it.

@abadams
abadams merged commit 994feba into main Sep 16, 2026
22 checks passed
@abadams abadams removed the dev_meeting Topic to be discussed at the next dev meeting label Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip_buildbots Do not run buildbots on this PR. Must add before opening PR as we scan labels immediately.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants