Skip to content

Changes to remove build steps for PE formatted guests. #486

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

Conversation

simongdavies
Copy link
Contributor

@simongdavies simongdavies commented May 13, 2025

Now that we have removed support for PE formatted guests (#470), we no longer need to build/archive these guests.

This completes the work for issue #470

@simongdavies simongdavies added kind/enhancement For PRs adding features, improving functionality, docs, tests, etc. area/infrastructure Concerns infrastructure rather than core functionality labels May 13, 2025
@simongdavies simongdavies enabled auto-merge (squash) May 13, 2025 08:16
@simongdavies simongdavies force-pushed the update-ci-and-build-for-pe-guest-removal branch 2 times, most recently from f4f7666 to eceecd1 Compare May 13, 2025 08:35
@simongdavies simongdavies force-pushed the update-ci-and-build-for-pe-guest-removal branch from eceecd1 to a3525b5 Compare May 13, 2025 08:46
Copy link
Contributor

@dblnz dblnz left a comment

Choose a reason for hiding this comment

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

LGTM

@simongdavies simongdavies merged commit d3a7498 into hyperlight-dev:main May 13, 2025
57 of 71 checks passed
@ludfjig
Copy link
Contributor

ludfjig commented May 13, 2025

@simongdavies there is still some PE info in the cargo/config.toml for each guest that we can probably remove

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infrastructure Concerns infrastructure rather than core functionality kind/enhancement For PRs adding features, improving functionality, docs, tests, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants