Skip to content

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

Open
ericapisani wants to merge 2 commits into
masterfrom
ep/remove-attribution-rule
Open

docs: Remove AI commit attribution requirement from AGENTS.md#6828
ericapisani wants to merge 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?

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