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

WIP: Add Cirrus support #382

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

adrianriobo
Copy link
Collaborator

Fix #348

Depends on #377

Signed-off-by: Adrian Riobo <[email protected]>
previously init functions for context had bunch of parameters which were hard to set with default values, this commit includes a struct to hold all possible params, also the contxt initalization has been moved from cmds to actions improving the design of the solution, as cmd code should have no logic at all and act only as wrapper to invoke action

Signed-off-by: Adrian Riobo <[email protected]>
this commit will change several deatils around the mac-pool workflow and the expectation on how the values for the machine should be treated

Signed-off-by: Adrian Riobo <[email protected]>
This commit is a prepartion for target more integrations similar to ghactions, to do it we move the management for the integration to the context, now we also added an integrations package where new integrations can be added and also moved the init for the integrations to the logic context which allows to keep moving th reamining logic from cmd to the actions

Signed-off-by: Adrian Riobo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[enhancement] Mapt as CIRRUS persistent worker
1 participant