Skip to content

docs: Remove AI commit attribution requirement from AGENTS.md#6828

Merged
ericapisani merged 2 commits into
masterfrom
ep/remove-attribution-rule
Jul 23, 2026
Merged

docs: Remove AI commit attribution requirement from AGENTS.md#6828
ericapisani merged 2 commits into
masterfrom
ep/remove-attribution-rule

Conversation

@ericapisani

Copy link
Copy Markdown
Member

Drop the rule requiring a Co-Authored-By line for AI-generated commits.

Drop the rule requiring a Co-Authored-By line for AI-generated commits.
@ericapisani
ericapisani marked this pull request as ready for review July 15, 2026 19:59
@ericapisani
ericapisani requested a review from a team as a code owner July 15, 2026 19:59
@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

91914 passed | ⏭️ 6302 skipped | Total: 98216 | Pass Rate: 93.58% | Execution Time: 314m 6s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests 📈 +1
Failed Tests 📉 -1
Skipped Tests

➖ Removed Tests (1)

View removed tests
  • test_circular_references
    • File: tests.tracing.test_misc

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2478 uncovered lines.
✅ Project coverage is 89.68%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.68%    89.68%        —%
==========================================
  Files          193       193         —
  Lines        24007     24007         —
  Branches      8342      8342         —
==========================================
+ Hits         21529     21529         —
- Misses        2478      2478         —
- Partials      1387      1387         —

Generated by Codecov Action

@cleptric

Copy link
Copy Markdown
Member

We actually would prefer if we can attribute commits to a certain model. If you use sentry-skill:commit this works out of the box.

@ericapisani

Copy link
Copy Markdown
Member Author

If you use sentry-skill:commit this works out of the box.

I think my version of this skill may have been out of date so I'll see what happens now that I've refreshed it 👀

We actually would prefer if we can attribute commits to a certain model.

Can you provide context on why? What are we hoping to do with this information? Do we have any dashboards, etc. that's tied to this info?

@ericapisani
ericapisani merged commit 8027343 into master Jul 23, 2026
146 checks passed
@ericapisani
ericapisani deleted the ep/remove-attribution-rule branch July 23, 2026 13:18
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.

3 participants