-
Notifications
You must be signed in to change notification settings - Fork 173
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
feat: package generation ALPHA #2269
Conversation
✅ Deploy Preview for zarf-docs canceled.
|
I was able to smoke test function pre and post rebase via:
Both produced zarf.yaml
|
a few things to chase down/confirm:
a few things I know can be improved:
|
28393f0
to
eda795e
Compare
1ea909d
to
bf8b450
Compare
5295c12
to
a375b26
Compare
LGTM, we will need approval from someone else on the zarf team |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding comments, mostly cleaning up string building, but there are a few things that could make this more generally useful and one issue with garbled text.
4cc0aad
to
5260a37
Compare
Signed-off-by: razzle <[email protected]>
Signed-off-by: razzle <[email protected]>
Signed-off-by: razzle <[email protected]>
Signed-off-by: razzle <[email protected]>
Signed-off-by: razzle <[email protected]>
Description
Introduce Zarf package generation from a source
Related Issue
Fixes #821
Type of change
Checklist before merging