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

Feature/Swift 5.5 Supports #72

Merged
merged 2 commits into from
Nov 15, 2021

Conversation

RocZhang9673
Copy link
Contributor

This will cause confusion and conflict with freshOS/Then 's global function await.

Users will get compiler errors if they build freshOS/Then with Swift 5.5. Although they still can use Then.await to avoid naming conflict, but it's not very convenient.

  • Changes
    • Rename await to awaitPromise.
    • Add one await function (which will call awaitPromise actually) with deprecated mark for short term back-port supports.

All unit test cases passed on my machine, with Xcode 13 Beta 3.

@hakocat
Copy link

hakocat commented Aug 8, 2021

LGTM

@cillyfly
Copy link

LGTM

2 similar comments
@yuhanle
Copy link

yuhanle commented Sep 23, 2021

LGTM

@mgontar-sciforce
Copy link

LGTM

wooster pushed a commit to modmed/then that referenced this pull request Oct 1, 2021
@vimask
Copy link

vimask commented Oct 5, 2021

LGTM

@dasoga
Copy link

dasoga commented Oct 7, 2021

Are you planning to merge this soon??

@cillyfly cillyfly mentioned this pull request Oct 8, 2021
@jeannchuab
Copy link

What is the forecast to merge this?

@julestburt
Copy link

LGTM

1 similar comment
@patrikbelis
Copy link

LGTM

@s4cha s4cha merged commit 914102e into freshOS:master Nov 15, 2021
@s4cha
Copy link
Member

s4cha commented Nov 15, 2021

This is now merged and live in https://github.com/freshOS/Then/releases/tag/6.0.0, thank you all for your undefeatable patience 😅

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.