Skip to content

chore(release): install cross before injector build#3493

Merged
AustinAbro321 merged 1 commit intomainfrom
install-cross
Feb 13, 2025
Merged

chore(release): install cross before injector build#3493
AustinAbro321 merged 1 commit intomainfrom
install-cross

Conversation

@AustinAbro321
Copy link
Copy Markdown
Member

Description

We reworked the injector process to use cross, but didn't install cross before building

Checklist before merging

@AustinAbro321 AustinAbro321 requested review from a team as code owners February 12, 2025 14:35
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 12, 2025

Deploy Preview for zarf-docs ready!

Name Link
🔨 Latest commit da596f3
🔍 Latest deploy log https://app.netlify.com/sites/zarf-docs/deploys/67acb1f48b71ad000846d0bb
😎 Deploy Preview https://deploy-preview-3493--zarf-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Austin Abro <AustinAbro321@gmail.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

@AustinAbro321
Copy link
Copy Markdown
Member Author

Build was successful - https://github.com/zarf-dev/zarf/actions/runs/13287839378/job/37100764939 (job failed because only main is allowed to publish to aws)

@AustinAbro321 AustinAbro321 added this pull request to the merge queue Feb 13, 2025
Merged via the queue into main with commit e7fcd78 Feb 13, 2025
@AustinAbro321 AustinAbro321 deleted the install-cross branch February 13, 2025 01:05
nevinaragam pushed a commit to nevinaragam/zarf that referenced this pull request May 20, 2025
Signed-off-by: Austin Abro <AustinAbro321@gmail.com>
Signed-off-by: NevinAragam <nevin.aragam@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants