-
Notifications
You must be signed in to change notification settings - Fork 246
Add async and await support to DaysUntilBirthday Swift sample app #191
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
base: main
Are you sure you want to change the base?
Conversation
|
Could you rebase this branch against |
c485819 to
b82ac43
Compare
|
Yep! Sorry about that. |
Samples/Swift/DaysUntilBirthday/DaysUntilBirthday.xcodeproj/project.pbxproj
Outdated
Show resolved
Hide resolved
Samples/Swift/DaysUntilBirthday/Shared/Services/BirthdayLoader.swift
Outdated
Show resolved
Hide resolved
Samples/Swift/DaysUntilBirthday/macOS/DaysUntilBirthdayOnMac.entitlements
Outdated
Show resolved
Hide resolved
Samples/Swift/DaysUntilBirthday/Shared/ViewModels/AuthenticationViewModel.swift
Outdated
Show resolved
Hide resolved
b82ac43 to
0bb46ee
Compare
0bb46ee to
db1677e
Compare
petea
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Discussed offline and agreed that we would wait to merge into main until we're closer to shipping Swift Concurrency support.
14368a9 to
a75b5ab
Compare
|
@mdmathias this is now in sync with |
36fb2dd to
d0deb73
Compare
No description provided.