Skip to content
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

Add package publish #1570

Merged
merged 25 commits into from
Sep 9, 2024
Merged

Add package publish #1570

merged 25 commits into from
Sep 9, 2024

Conversation

THenry14
Copy link
Member

@THenry14 THenry14 commented Sep 2, 2024

This PR adds a support for publishing packages to a registry.
In order to succesfully upload a package one needs to provide an authentication token via SCARB_AUTH_TOKEN env variable.

sncast_std $ cargo run --manifest-path /path/to/scarb/Cargo.toml --bin scarb -- publish --index http://localhost:8000/ 
   Packaging sncast_std v0.30.0 (/(...)/sncast_std/Scarb.toml)
   Verifying sncast_std-0.30.0.tar.zst
   Compiling sncast_std v0.30.0 (/(...)/sncast_std/target/package/sncast_std-0.30.0/Scarb.toml)
    Finished release target(s) in 33 seconds
    Packaged 7 files, 14.10 KiB (3.71 KiB compressed)
   Uploading sncast_std v0.30.0 (registry+http://localhost:8000/)
   Published sncast_std v0.30.0 (registry+http://localhost:8000/)

@THenry14 THenry14 changed the title [Draft] Add package publish [WIP] Add package publish Sep 2, 2024
@THenry14 THenry14 force-pushed the szymczyk/publish branch 25 times, most recently from 48d2074 to a776a49 Compare September 5, 2024 11:45
@THenry14 THenry14 changed the title [WIP] Add package publish Add package publish Sep 5, 2024
@THenry14 THenry14 marked this pull request as ready for review September 5, 2024 12:16
scarb/src/flock.rs Outdated Show resolved Hide resolved
@THenry14 THenry14 requested a review from mkaput September 6, 2024 11:52
website/docs/registries/publishing.md Show resolved Hide resolved
website/docs/registries/publishing.md Outdated Show resolved Hide resolved
scarb/src/core/registry/client/http.rs Outdated Show resolved Hide resolved
scarb/src/core/registry/client/http.rs Outdated Show resolved Hide resolved
scarb/src/core/registry/client/http.rs Outdated Show resolved Hide resolved
scarb/src/core/registry/client/http.rs Outdated Show resolved Hide resolved
@THenry14 THenry14 requested a review from DelevoXDG September 6, 2024 12:20
Copy link
Member

@mkaput mkaput left a comment

Choose a reason for hiding this comment

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

lgtm from my side, though I didn't review this carefully (only picked stuff that interested me)

@THenry14 THenry14 added this pull request to the merge queue Sep 9, 2024
Merged via the queue into main with commit da11cab Sep 9, 2024
22 checks passed
@THenry14 THenry14 deleted the szymczyk/publish branch September 9, 2024 08:49
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.

4 participants