Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates client to swift concurrency and dependency key #5

Merged
merged 11 commits into from
Nov 7, 2022

Conversation

Miiha
Copy link
Owner

@Miiha Miiha commented Nov 5, 2022

Updated the library to incorporate the latest improvements to TCA.

@Miiha Miiha requested a review from andreyz November 5, 2022 16:50
@andreyz
Copy link
Collaborator

andreyz commented Nov 5, 2022

Hi @Miiha, thanks for requesting the review. I'll read through the diff and comment if needed by Monday.

Really great taking up effort to modernize the library in line with latest TCA.

@Miiha
Copy link
Owner Author

Miiha commented Nov 5, 2022

That sounds awesome @andreyz

Copy link
Collaborator

@andreyz andreyz left a comment

Choose a reason for hiding this comment

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

There're minor improvements suggested in comments.
Great job on the refresh and the package itself!

Examples/Example/Example/ContentView.swift Outdated Show resolved Hide resolved
Sources/ComposableUserNotifications/TestKey.swift Outdated Show resolved Hide resolved
Sources/ComposableUserNotifications/TestKey.swift Outdated Show resolved Hide resolved
Sources/ComposableUserNotifications/LiveKey.swift Outdated Show resolved Hide resolved
Examples/Example/Example/App.swift Outdated Show resolved Hide resolved
Sources/ComposableUserNotifications/Interface.swift Outdated Show resolved Hide resolved
Sources/ComposableUserNotifications/LiveKey.swift Outdated Show resolved Hide resolved
@Miiha
Copy link
Owner Author

Miiha commented Nov 6, 2022

Thanks @andreyz for your comments and for taking the time!

@Miiha Miiha changed the title Updates to reducer protocol and dependency keys Updates client to swift concurrency and dependency key Nov 7, 2022
@Miiha Miiha merged commit 94b1a3b into main Nov 7, 2022
@Miiha Miiha deleted the reducer-protocol-plus-dependency-update branch November 7, 2022 17:36
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