Skip to content

Disfavor deprecated init.#69

Merged
mbrandonw merged 2 commits into
mainfrom
fix-ambiguous-init
May 27, 2025
Merged

Disfavor deprecated init.#69
mbrandonw merged 2 commits into
mainfrom
fix-ambiguous-init

Conversation

@mbrandonw
Copy link
Copy Markdown
Member

With #63 we created the potential for some ambiguous inits. This does not currently work:

@FetchAll(Reminder.order(by: \.isCompleted), animation: .default)
var reminders: [Reminder]

So I am disfavoring the deprecated init to make this pick the right one.

@mbrandonw mbrandonw requested a review from stephencelis May 27, 2025 16:15
@mbrandonw mbrandonw merged commit 27aa920 into main May 27, 2025
3 checks passed
@mbrandonw mbrandonw deleted the fix-ambiguous-init branch May 27, 2025 16:37
mackoj pushed a commit to mackoj/sqlite-data that referenced this pull request Dec 5, 2025
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.

1 participant