Skip to content

Fix crates.io publish failure: rename package to vrillabs-helios#16

Merged
VrilLabs merged 1 commit into
mainfrom
copilot/fix-publish-helios-job-again
Jul 2, 2026
Merged

Fix crates.io publish failure: rename package to vrillabs-helios#16
VrilLabs merged 1 commit into
mainfrom
copilot/fix-publish-helios-job-again

Conversation

Copilot AI commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

The "Publish helios to crates.io" job was failing because publish-crate.yml already referenced the crates.io package name vrillabs-helios (required since helios is already published by another author), but rs/helios/Cargo.toml still declared name = "helios", causing cargo metadata to fail with vrillabs-helios package not found in cargo metadata output.

Package rename

  • Changed [package].name in rs/helios/Cargo.toml from helios to vrillabs-helios; binary and lib crate names remain helios, so the CLI and import paths are unaffected.
  • Updated rs/Cargo.lock to match the renamed package, repositioned alphabetically, with no dependency version changes.

Workflow fixes

  • publish-crate.yml: cargo update -p helioscargo update -p vrillabs-helios.
  • release.yml: cargo build --release --locked -p helios-p vrillabs-helios (this selector also targets the package by name, not the binary).
# rs/helios/Cargo.toml
[package]
name = "vrillabs-helios"   # was "helios"

@VrilLabs VrilLabs marked this pull request as ready for review July 2, 2026 10:29
Copilot AI review requested due to automatic review settings July 2, 2026 10:29
@VrilLabs VrilLabs merged commit 5d9c252 into main Jul 2, 2026
2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

3 participants