Skip to content

Conversation

ludfjig
Copy link
Contributor

@ludfjig ludfjig commented Dec 12, 2024

These environment KVM_SHOULD_BE_PRESENT HYPERV_SHOULD_BE_PRESENT variables just add unnecessary complexity, especially for newcomers who just cloned the repo. They're not very useful anyway

@ludfjig ludfjig added the kind/enhancement For PRs adding features, improving functionality, docs, tests, etc. label Dec 12, 2024
@ludfjig ludfjig marked this pull request as ready for review December 12, 2024 18:24
@ludfjig ludfjig requested a review from simongdavies December 12, 2024 21:15
Copy link
Contributor

@simongdavies simongdavies left a comment

Choose a reason for hiding this comment

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

One small nit

simongdavies
simongdavies previously approved these changes Jan 16, 2025
@simongdavies
Copy link
Contributor

@ludfjig once this merges we should update the common workflow setup to get rid of the variables there too

dblnz
dblnz previously approved these changes Jan 20, 2025
@ludfjig
Copy link
Contributor Author

ludfjig commented Jan 23, 2025

@ludfjig once this merges we should update the common workflow setup to get rid of the variables there too

hyperlight-dev/ci-setup-workflow#4

@ludfjig ludfjig force-pushed the remove_env_test branch 2 times, most recently from 30893ab to b59e40d Compare January 23, 2025 23:03
@ludfjig ludfjig requested a review from simongdavies January 28, 2025 19:20
@ludfjig ludfjig merged commit 288d6f9 into hyperlight-dev:main Jan 28, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement For PRs adding features, improving functionality, docs, tests, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants