FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

docs: Remove AI commit attribution requirement from AGENTS.md by ericapisani · Pull Request #6828 · getsentry/sentry-python · GitHub

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

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 marked this pull request as ready for review July 15, 2026 19:59
ericapisani requested a review from a team as a code owner July 15, 2026 19:59

github-actions Bot commented Jul 15, 2026
edited
Loading

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

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.

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 merged commit 8027343 into master Jul 23, 2026
146 checks passed
ericapisani deleted the ep/remove-attribution-rule branch July 23, 2026 13:18
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
…try#6828)

Drop the rule requiring a Co-Authored-By line for AI-generated commits.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL