Skip to content

Conversation

@roanutil
Copy link
Owner

I think our changes for v3.0 are stable enough. The API has been pretty stable and our use of the library has grown quite a bit.

roanutil added 30 commits June 12, 2023 23:18
- Replace publisher based subscriptions with AsyncStream and AsyncThrowingStream
- Add subscriptions for aggregate functions
- Change CoreDataRepositoryError.coreData to hold CocoaError instead of NSError
- Add CoreDataRepositoryError.unknown that holds a NSError as a fallback
- Guard against invalid descriptions in aggregate functions to avoid runtime panics
- Add type hint parameters to many functions to avoid the need to fully qualify the return signature at the call site
- Remove most existing doc comments which will be re-written soon
- Simplify aggregate return type to only the desired value
….coreData to CoreDataError.cocoa

3.0-preview
3.0-preview
3.0-preview
3.0-preview-async-deadlock
3.0-preview-throwing-unmanaged-model-init
remove-unstructured-tasks-from-subscription
remove-unstructured-tasks-from-subscription-alt
remove-unstructured-tasks-from-subscription-alt
remove-unstructured-tasks-from-subscription-alt
remove-unstructured-tasks-from-subscription-alt
remove-unstructured-tasks-from-subscription-alt
# Conflicts:
#	Package.resolved
roanutil and others added 17 commits June 13, 2025 17:03
feature/swift-testing
…r and tests for CoreDataTestSuite

feature/swift-testing
feature/swift-testing
…before exiting withTransaction

feature/transactions
Feature/transactions (Depends on PR #37)
…fiedUnmanagedModel

feature/add-unmanagedId-property
feature/add-unmanagedId-property
feature/fix-misnamed-property

# Conflicts:
#	Sources/CoreDataRepository/IdentifiedUnmanagedModel.swift
#	Sources/Internal/ModelsWithIntId/IdentifiableModel_Int.swift
#	Sources/Internal/ModelsWithUuidId/IdentifiableModel_Uuid.swift
feature/fix-misnamed-property
Remove unmanagedIdAccessor in favor of unmanagedId property in IdentifiedUnmanagedModel
# Conflicts:
#	Sources/CoreDataRepository/IdentifiedUnmanagedModel.swift
#	Sources/Internal/ModelsWithIntId/IdentifiableModel_Int.swift
#	Sources/Internal/ModelsWithUuidId/IdentifiableModel_Uuid.swift
Fix name of IdentifiedUnmanagedModel.unmanagedIdExpression
bugfix/build-errors-with-swift-6-2
bugfix/build-errors-with-swift-6-2
@codecov-commenter
Copy link

codecov-commenter commented Nov 11, 2025

Codecov Report

❌ Patch coverage is 84.40391% with 1180 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.43%. Comparing base (c77247f) to head (90b2c31).

Files with missing lines Patch % Lines
.../Internal/ModelsWithIntId/ManagedIdModel_Int.swift 0.00% 101 Missing ⚠️
...ternal/ModelsWithIntId/ManagedIdUrlModel_Int.swift 0.00% 101 Missing ⚠️
Sources/CoreDataRepository/CoreDataError.swift 30.08% 86 Missing ⚠️
...nternal/ModelsWithUuidId/UnmanagedModel_Uuid.swift 0.00% 79 Missing ⚠️
.../Internal/ModelsWithIntId/UnmanagedModel_Int.swift 0.00% 67 Missing ⚠️
...ternal/ModelsWithIntId/IdentifiableModel_Int.swift 0.00% 61 Missing ⚠️
.../Internal/ModelsWithIntId/FetchableModel_Int.swift 0.00% 55 Missing ⚠️
Sources/Internal/CoreDataStack.swift 45.00% 44 Missing ⚠️
...ataRepository/Internal/AggregateSubscription.swift 33.84% 43 Missing ⚠️
...itory/Internal/AggregateThrowingSubscription.swift 33.84% 43 Missing ⚠️
... and 46 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
- Coverage   86.09%   84.43%   -1.67%     
==========================================
  Files          20       69      +49     
  Lines        1683     7587    +5904     
==========================================
+ Hits         1449     6406    +4957     
- Misses        234     1181     +947     

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

@r-jarvis
Copy link
Collaborator

I definitely agree with this! It will be great to be able to use standard tag/release for package manager moving forward. Congrats on finishing up this version!

@r-jarvis r-jarvis merged commit d7b90d1 into main Nov 26, 2025
2 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.

4 participants