FCM icon indicating copy to clipboard operation
FCM copied to clipboard

Support for Swift6 and async/await

Open gennaro-safehill opened this issue 1 year ago • 4 comments

gennaro-safehill avatar Sep 30 '24 23:09 gennaro-safehill

Hey @gennaro-safehill, thanks for the great pull request! It looks good. Is it ready to be merged?

MihaelIsaev avatar Oct 01 '24 05:10 MihaelIsaev

Hey @gennaro-safehill, thanks for the great pull request! It looks good. Is it ready to be merged?

You're welcome! I'm an open-sourcer and it's all about helping the community! I'm testing the code out right now, I'll let you know

gennaro-safehill avatar Oct 02 '24 00:10 gennaro-safehill

Thank you @gennaro-safehill, once you confirm it works well I'm ready to merge and tag it as v3.0.0

MihaelIsaev avatar Oct 02 '24 18:10 MihaelIsaev

I run into some problems deploying this code to Heroku using Swift 6. There are some problems with the buildpack.

Using Swift 5.10 works. There are no code changes, it's only about the swift-tools-version in Package.swift. Testing the changes right now in staging.

gennaro-safehill avatar Oct 02 '24 21:10 gennaro-safehill

Hi folks. Just wanted to let you know that I wasn't able to test these changes in my production environment with Swift 6 because we use Heroku and the buildpack for Swift 6 has some compat issues.

I did test them with 5.10, and they work great! We have 2 options:

  1. Merge this code as is
  2. Merge the 5.10 version at https://github.com/MihaelIsaev/FCM/pull/49 (there's only one line change on top of the changes proposed here setting the swift-tools-version to 5.10.0

Let me know what you prefer

gennaro-safehill avatar Oct 21 '24 16:10 gennaro-safehill

Hey, would love to update to JWTKit 5 too and therefore relying on this. Are there any blockers?

ptoffy avatar Nov 07 '24 15:11 ptoffy

No blockers. See my message above

gennaro-safehill avatar Nov 07 '24 20:11 gennaro-safehill

@gennaro-safehill I think this will work great! I'm ready to merge and tag it as v3 once the README is updated. Could you please update the README as well?

MihaelIsaev avatar Nov 08 '24 01:11 MihaelIsaev

@gennaro-safehill Many thanks for your contribution! It is now available since 3.0.0-beta.1 tag.

MihaelIsaev avatar Nov 11 '24 16:11 MihaelIsaev