Skip to content
This repository was archived by the owner on Jul 4, 2020. It is now read-only.

Change ActionContainer Array to be ActionContainer #7

Closed
wants to merge 2 commits into from

Conversation

a2
Copy link
Owner

@a2 a2 commented Dec 24, 2018

This PR removes the + operator in favor of joining Actions in Arrays.

Before:

func bar() -> ActionContainer { ... }
let shortcut = buildShortcut(foo() + bar() + baz())

After:

let bars: ActionContainer = [bar(), bar(), bar()]
let shortcut = buildShortcut([foo(), bars, baz()])

Fixes #3

@a2 a2 requested a review from alexaubry December 24, 2018 12:27
@a2
Copy link
Owner Author

a2 commented Dec 24, 2018

This doesn't actually work because of type constraints:

Using 'ActionContainer' as a concrete type conforming to protocol 'ActionContainer' is not supported

@a2 a2 closed this Dec 24, 2018
@a2 a2 deleted the recursive-actioncontainer branch December 24, 2018 12:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant