insidegui / WWDC

The unofficial WWDC app for macOS
https://wwdc.io
BSD 2-Clause "Simplified" License
8.6k stars 777 forks source link

Remove RxSwift in favor of combine #671

Closed allenhumphreys closed 1 year ago

allenhumphreys commented 1 year ago
insidegui commented 1 year ago

This is way smaller than I thought it would be 😅

We'll definitely wait until after next week's keynote before merging this, since we already have a lengthy PR (#669) for this year's update, but I'm definitely in favor of doing it.

allenhumphreys commented 1 year ago

@insidegui I think this should probably get merged in so you can start building on it too. I don't think it's perfect, but there's some good fixes in here.

I cannot test Touch Bar or CloudKit stuff, so there's a few places that are #if 'd away that may need to be migrated to Combine!

insidegui commented 1 year ago

@allenhumphreys I don't have Touch Bar hardware either, but there's a simulator in Xcode. Window > Touch Bar > Show Touch Bar. Could you test with that and let me know?

I'll check out the CloudKit stuff.

allenhumphreys commented 1 year ago

Could you test with that and let me know?

@insidegui It's still working as expected, I think the icons will get an update when you finish your branch, so remember to check the Touch Bar buttons in that branch!

insidegui commented 1 year ago

I wonder if we should even keep Touch Bar support in. It’s not trivial to support and Apple has made it quite clear that it’s a dead feature.

allenhumphreys commented 1 year ago

If the now playing stuff gives the basic controls in the touchbar... seems like that could make sense. On the other hand there are a lot of touchbar macs out there.