Skip to content

Warn on complex altkey types - #50

Merged
MarcusRisanger merged 2 commits into
mainfrom
warn-on-complex-altkey-types
Dec 1, 2025
Merged

Warn on complex altkey types#50
MarcusRisanger merged 2 commits into
mainfrom
warn-on-complex-altkey-types

Conversation

@MarcusRisanger

Copy link
Copy Markdown
Owner

Closes #48 .

  • This fix adds a type checker for alternate key columns
  • If complex types are found in the columns that comprise the alternate key, the user is warned that these should ideally be serialized prior to calling upsert.

@MarcusRisanger MarcusRisanger self-assigned this Dec 1, 2025
@codecov

codecov Bot commented Dec 1, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.67%. Comparing base (1e92c53) to head (bf4a18e).

Additional details and impacted files
@@            Coverage Diff            @@
##           main      #50       +/-   ##
=========================================
+ Coverage      0   91.67%   +91.67%     
=========================================
  Files         0       17       +17     
  Lines         0      901      +901     
=========================================
+ Hits          0      826      +826     
- Misses        0       75       +75     

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

@MarcusRisanger
MarcusRisanger merged commit 19208f3 into main Dec 1, 2025
8 checks passed
@MarcusRisanger
MarcusRisanger deleted the warn-on-complex-altkey-types branch December 4, 2025 19:03
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.

Raise on complex types in altkeys

1 participant