Skip to content
This repository has been archived by the owner on Mar 17, 2023. It is now read-only.

composite actions #32

Closed
frantjc opened this issue Feb 12, 2022 · 2 comments
Closed

composite actions #32

frantjc opened this issue Feb 12, 2022 · 2 comments
Assignees
@frantjc

Comments

@frantjc
Copy link
Owner

frantjc commented Feb 12, 2022 •
edited
Loading

currently, we map an action to 2 steps: 1 to get the action and its metadata, and another to execute it.

composite actions throw a wrench in that; we will need 1 step to get the action and its metadata, and an arbitrary number of steps to execute it
@frantjc frantjc mentioned this issue Mar 25, 2022
Closed
@frantjc
Copy link
Owner Author

frantjc commented Mar 26, 2022 •
edited
Loading

update: an action now maps to up to 3 steps (pre, main, post), but they're all non-uses steps. since composite actions can have uses steps, we almost need to recurse again....which is bad news

@frantjc
Copy link
Owner Author

frantjc commented Apr 4, 2022

haven't tested yet but I believe this is resolved as of #45

@frantjc frantjc self-assigned this Apr 4, 2022
@frantjc frantjc closed this as completed Apr 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@frantjc frantjc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@frantjc