Skip to content

feat: Add support for declaring triggers in __table_args__. - #150

Merged
DanCardin merged 1 commit into
DanCardin:mainfrom
Dan-Knott:dk/trigger-table-args
Sep 16, 2026
Merged

DanCardin merged 1 commit into
DanCardin:mainfrom
Dan-Knott:dk/trigger-table-args

Conversation

@Dan-Knott

Copy link
Copy Markdown
Contributor

I've always wanted to have triggers declared on tables just like how you do with Index and Constraint.

@codecov

codecov Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.92%. Comparing base (88b47a9) to head (09e5886).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
.../sqlalchemy_declarative_extensions/trigger/base.py 85.71% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
- Coverage   89.94%   89.92%   -0.03%     
==========================================
  Files          86       86              
  Lines        4136     4147      +11     
  Branches      863      866       +3     
==========================================
+ Hits         3720     3729       +9     
- Misses        337      338       +1     
- Partials       79       80       +1     

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

@DanCardin DanCardin left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, great idea!

@DanCardin
DanCardin merged commit 4a9bb9d into DanCardin:main Sep 16, 2026
17 of 19 checks passed
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