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

Combine Worker #104

Merged
merged 17 commits into from
Nov 3, 2021
Merged

Combine Worker #104

merged 17 commits into from
Nov 3, 2021

Conversation

soorinpark
Copy link
Collaborator

@soorinpark soorinpark commented Oct 28, 2021

Checklist

  • Unit Tests
  • UI Tests
  • Snapshot Tests (iOS only)
  • I have made corresponding changes to the documentation

@CLAassistant
Copy link

CLAassistant commented Oct 28, 2021

CLA assistant check
All committers have signed the CLA.

@soorinpark soorinpark force-pushed the soorin/combineWorker branch from 1e59af3 to 9f0b8fc Compare November 1, 2021 15:18
@soorinpark soorinpark force-pushed the soorin/combineWorker branch from f1c1a20 to ecf518d Compare November 1, 2021 16:52
@soorinpark soorinpark self-assigned this Nov 1, 2021
@soorinpark soorinpark changed the title [WIP] Combine Worker Combine Worker Nov 1, 2021
@soorinpark soorinpark force-pushed the soorin/combineWorker branch 2 times, most recently from 6465719 to eb089cc Compare November 2, 2021 22:08
@soorinpark soorinpark force-pushed the soorin/combineWorker branch from eb089cc to 2057460 Compare November 2, 2021 22:12
@soorinpark soorinpark force-pushed the soorin/combineWorker branch 2 times, most recently from 21c91dd to 1f971a8 Compare November 3, 2021 20:15
@soorinpark soorinpark force-pushed the soorin/combineWorker branch from 1f971a8 to 07fd80a Compare November 3, 2021 20:16
import Foundation
import Workflow

@available(iOS 13.0, macOS 10.15, *)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we need these for each iOS 13+ API or if we can require iOS 13 at the library level. 🤔 cc: @amorde

@soorinpark soorinpark force-pushed the soorin/combineWorker branch from 120f752 to c4d4d10 Compare November 3, 2021 21:12
Copy link
Contributor

@dhavalshreyas dhavalshreyas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳 🥂

This is awesome! Thank you @soorinpark!

@soorinpark soorinpark merged commit 50f3c3b into main Nov 3, 2021
@soorinpark soorinpark deleted the soorin/combineWorker branch November 3, 2021 21:24
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.

4 participants