-
Notifications
You must be signed in to change notification settings - Fork 98
Conversation
Of course, |
This requires changes to |
@MaxDesiatov Thanks for getting a head start on this! A lot of the work I'd been wanting to do on I just pushed a few commits to test Swift 5.2 and 5.3 on CI for Linux and macOS (I'd like to keep support for 5.2 in the short term, to give folks some time to upgrade). Edit Ah yikes, I didn't realize you were actively working on this, so apologies for meddling. I'll stay clear until this is ready to go. |
Thanks! This now depends on NSHipster/SwiftSyntaxHighlighter#8 |
Thanks for your help with this, @MaxDesiatov! Glad to have this blocker out of the way so that we can target Swift 5.3 for future releases. |
Thanks for finishing the PR! |
I'm not sure if you'd like to test with both Swift 5.2 and 5.3 on Linux. As you're using a cache, I think it would be easier to test just with a single version, as it doesn't make sense to share that cache. Let me know if you'd prefer to test both versions after all with separate cache keys.