Skip to content

Remove version number from hyperlight-testing crate dependency #588

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

Merged
merged 1 commit into from
Jun 9, 2025

Conversation

simongdavies
Copy link
Contributor

In #580 a version number was adding to the hyperlight-testing dependency. This caused the Cargo pubish job for the last release to fail.

This PR removes the version number so we don't have to publish the hyperlight-testing crate

@simongdavies simongdavies added the kind/bugfix For PRs that fix bugs label Jun 9, 2025
@simongdavies simongdavies enabled auto-merge (squash) June 9, 2025 08:43
@jprendes
Copy link
Contributor

jprendes commented Jun 9, 2025

I think we also need to remove the

version.workspace = true

from src/hyperlight_testing/Cargo.toml

@simongdavies
Copy link
Contributor Author

simongdavies commented Jun 9, 2025

I think we also need to remove the

version.workspace = true

from src/hyperlight_testing/Cargo.toml

How come, I did a dry run publish and it worked

@jprendes
Copy link
Contributor

jprendes commented Jun 9, 2025

#580 added the version to src/hyperlight_testing/Cargo.toml as well.
Removing it from the workspace might be enough to fix the issue, but I think we should also remove it from the crate's toml file.
If dry run is happy, then it's just a nit. I let you decide if you want to remove it now or not :-)

jprendes
jprendes previously approved these changes Jun 9, 2025
Copy link
Contributor

@jprendes jprendes left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
Copy link
Contributor

@jprendes jprendes 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 b0d395d into hyperlight-dev:main Jun 9, 2025
29 checks passed
simongdavies added a commit to simongdavies/hyperlight that referenced this pull request Jun 9, 2025
…light-dev#588)

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
simongdavies added a commit that referenced this pull request Jun 9, 2025
…#589)

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bugfix For PRs that fix bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants