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

slicer/test: Set archive options from release #82

Merged
merged 1 commit into from
Sep 17, 2023

Conversation

woky
Copy link
Contributor

@woky woky commented Jun 21, 2023

Currently testArchive.Options() returns archive.Options with only the
Arch field set. In future we would like to extend testArchive to behave
differently based on its chisel.yaml definition.

Initialize archive.Options from setup.Release and store it in the
testArchive structure for each archive defined in chise.yaml.

@woky
Copy link
Contributor Author

woky commented Jun 21, 2023

This PR depends on #81

@woky woky added the Blocked Waiting for something external label Jun 21, 2023
Currently testArchive.Options() returns archive.Options with only the
Arch field set. In future we would like to extend testArchive to behave
differently based on its chisel.yaml definition.

Initialize archive.Options from setup.Release and store it in the
testArchive structure for each archive defined in chise.yaml.
@woky woky force-pushed the pub/slicer-test-archives-options branch from 9aad663 to 9cae335 Compare August 29, 2023 15:28
@woky woky added Priority Look at me first and removed Blocked Waiting for something external labels Aug 29, 2023
Copy link
Member

@rebornplusplus rebornplusplus left a comment

Choose a reason for hiding this comment

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

LGTM. You could probably add the Simple label to this one.

@woky woky added the Simple Nice for a quick look on a minute or two label Sep 11, 2023
Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

lgtm, ty

@flotter
Copy link

flotter commented Sep 14, 2023

Currently testArchive.Options() returns archive.Options with only the Arch field set. In future we would like to extend testArchive to behave differently based on its chisel.yaml definition.

Initialize archive.Options from setup.Release and store it in the testArchive structure for each archive defined in chise.yaml.

Just for incase you plan to use this in the commit, the last file name has a typo.

Copy link

@flotter flotter left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@niemeyer niemeyer left a comment

Choose a reason for hiding this comment

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

LGTM

@niemeyer niemeyer merged commit d4122cf into canonical:main Sep 17, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority Look at me first Simple Nice for a quick look on a minute or two
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants