Skip to content
This repository has been archived by the owner on Dec 16, 2024. It is now read-only.

Abstract a bit the uki commands #29

Closed
wants to merge 4 commits into from
Closed

Abstract a bit the uki commands #29

wants to merge 4 commits into from

Conversation

Itxaka
Copy link
Member

@Itxaka Itxaka commented Jan 23, 2024

Make use of the config with the runners and vfs so we can easily tests the actions being taken and substitute those structs for others

Make use of the config with the runners and vfs so we can easily tests
the actions being taken and substitute those structs for others

Signed-off-by: Itxaka <itxaka@kairos.io>
@@ -457,64 +456,25 @@ func copyFilesToImg(imgFile string, filesMap map[string][]string) error {
return nil
}

func GzipFile(sourcePath, targetPath string) error {
Copy link
Member Author

Choose a reason for hiding this comment

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

moved to utils

return nil
}

func findKairosVersion(sourceDir string) (string, error) {
Copy link
Member Author

Choose a reason for hiding this comment

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

same

}
}
})
It("Builds UKI Iso", func() {})
Copy link
Member Author

Choose a reason for hiding this comment

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

nothing yet added

Copy link
Contributor

Choose a reason for hiding this comment

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

Happening in this PR or other? Thanks for writing this btw, I looked at the other test and I was too afraid to try this :D.

Copy link
Member Author

Choose a reason for hiding this comment

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

Hopefully in this one lmao

@Itxaka Itxaka marked this pull request as ready for review January 25, 2024 13:16
@Itxaka Itxaka requested a review from a team January 25, 2024 13:16
@jimmykarily
Copy link
Contributor

Nice job!

@Itxaka Itxaka closed this Jul 11, 2024
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.

2 participants