Skip to content

feat: add simple AsFormField impl for ForeignKeys#335

Merged
m4tx merged 8 commits into
masterfrom
foreign-key-form-field
May 25, 2025
Merged

feat: add simple AsFormField impl for ForeignKeys#335
m4tx merged 8 commits into
masterfrom
foreign-key-form-field

Conversation

@m4tx

@m4tx m4tx commented May 20, 2025

Copy link
Copy Markdown
Member

No description provided.

@m4tx m4tx requested a review from seqre May 20, 2025 00:10
@m4tx m4tx changed the title feat: add simple AsFormField impl for ForeignKeys feat: add simple AsFormField impl for ForeignKeys May 20, 2025
@github-actions github-actions Bot added the C-lib Crate: cot (main library crate) label May 20, 2025
@codecov

codecov Bot commented May 20, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
rust 88.16% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
cot/src/form/fields.rs 87.67% <100.00%> (+0.30%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@m4tx m4tx force-pushed the foreign-key-form-field branch from 896307e to b61e73b Compare May 20, 2025 10:26
@github-actions github-actions Bot added the C-macros Crate: cot-macros label May 20, 2025
@m4tx m4tx changed the base branch from master to more-form-fields May 20, 2025 10:26

@seqre seqre left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good enough for now, but please add more tests

Base automatically changed from more-form-fields to master May 24, 2025 10:11
@m4tx m4tx enabled auto-merge (squash) May 25, 2025 09:07
@m4tx m4tx merged commit f9d7c43 into master May 25, 2025
22 checks passed
@m4tx m4tx deleted the foreign-key-form-field branch May 25, 2025 09:13
@cotbot cotbot Bot mentioned this pull request May 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-lib Crate: cot (main library crate) C-macros Crate: cot-macros

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants