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

refactor: normal creator #3114

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

refactor: normal creator #3114

wants to merge 1 commit into from

Conversation

phillebaba
Copy link
Member

@phillebaba phillebaba commented Oct 17, 2024

Description

This change refactors the package creation.

Related Issue

Relates to #2969

Checklist before merging

Copy link

netlify bot commented Oct 17, 2024

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit 49f5112
🔍 Latest deploy log https://app.netlify.com/sites/zarf-docs/deploys/67573c2b9f7eb70008f64242

@phillebaba phillebaba force-pushed the refactor/creator-normal branch from ae6c1dc to 6dbbc6b Compare October 17, 2024 13:41
Copy link

codecov bot commented Oct 17, 2024

@phillebaba phillebaba force-pushed the refactor/creator-normal branch 7 times, most recently from 47c5dd1 to 5b54240 Compare October 25, 2024 09:00
@phillebaba phillebaba force-pushed the refactor/creator-normal branch 9 times, most recently from cbd05e3 to ca69d35 Compare November 2, 2024 15:49
@phillebaba phillebaba force-pushed the refactor/creator-normal branch 6 times, most recently from 3c36c9d to 4a829c6 Compare November 5, 2024 21:57
@phillebaba phillebaba marked this pull request as ready for review November 5, 2024 21:58
@phillebaba phillebaba requested review from a team as code owners November 5, 2024 21:58
@phillebaba phillebaba force-pushed the refactor/creator-normal branch from 4a829c6 to 8cd0d1c Compare November 5, 2024 22:06
for idx, component := range pkg.Components {
pkg.Components[idx], _ = deprecated.MigrateComponent(pkg.Build, component)
}
// TODO (phillebaba): Figure out when migrations should actually be run.
Copy link
Member Author

Choose a reason for hiding this comment

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

Address this.

@phillebaba phillebaba force-pushed the refactor/creator-normal branch from 8cd0d1c to 673af5f Compare November 5, 2024 22:29
Copy link
Contributor

@AustinAbro321 AustinAbro321 left a comment

Choose a reason for hiding this comment

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

Great work, love that we are no longer changing directory. Left a few comments

src/cmd/package.go Show resolved Hide resolved
src/internal/packager2/actions/actions.go Outdated Show resolved Hide resolved
src/internal/packager2/layout/create.go Outdated Show resolved Hide resolved
src/internal/packager2/layout/create_test.go Outdated Show resolved Hide resolved
src/internal/packager2/layout/sbom_test.go Outdated Show resolved Hide resolved
@phillebaba phillebaba force-pushed the refactor/creator-normal branch 11 times, most recently from 60a658b to dcd8b19 Compare November 26, 2024 09:13
@phillebaba phillebaba force-pushed the refactor/creator-normal branch 4 times, most recently from 805199a to 8bf8990 Compare December 3, 2024 17:45
@AustinAbro321
Copy link
Contributor

Generally looks good. I do request that we keep the Package saved to "zarf-init-amd64-v0.43.0.tar.zst" message we have currently.

I know we're trying to limit output for non errors, but I think the package tar path is worth logging since the filename isn't intuitive. It's a combination of whether it's an init or package config, amd or arm64, and the version. As a user I've found it useful to copy + paste the name from terminal when my next step is to publish or deploy, especially when I have multiple packages in the same directory.

@phillebaba phillebaba force-pushed the refactor/creator-normal branch 2 times, most recently from 08cf981 to ad7f12a Compare December 9, 2024 18:01
Signed-off-by: Philip Laine <[email protected]>
@phillebaba phillebaba force-pushed the refactor/creator-normal branch from ad7f12a to 49f5112 Compare December 9, 2024 18:51
@phillebaba
Copy link
Member Author

I have now added back the logging for the path the package is saved to.

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.

2 participants