search

KeisukeYamashita / commitlint-rs

🔦 Lint commit messages with conventional commit messages
https://keisukeyamashita.github.io/commitlint-rs
Apache License 2.0
32 stars 8 forks source link

chore(other): enable cargo binstall #303

Closed slowsage closed 5 months ago

slowsage commented 5 months ago

Why

This is needed since the crate name and archive prefix differ. Related: #45

KeisukeYamashita commented 5 months ago

Thank you very much for your PR, let me check this on the weekend!

slowsage commented 5 months ago

Weird. For me, I get:

cargo binstall  commitlint-rs@0.1.9 --force 
 INFO resolve: Resolving package: 'commitlint-rs@=0.1.9'
 WARN The package commitlint-rs v0.1.9 will be installed from source (with cargo)
  Downloaded commitlint-rs v0.1.9
  Downloaded 1 crate (25.0 KB) in 0.04s
    Updating crates.io index
  Installing commitlint-rs v0.1.9
    Updating crates.io index
   Compiling proc-macro2 v1.0.82
   Compiling unicode-ident v1.0.12
   Compiling autocfg v1.3.0
   Compiling libc v0.2.154
   Compiling memchr v2.7.2
   Compiling futures-sink v0.3.30

cargo-binstall v1.6.5

Not sure if it downloaded the binary directly from your log. The below line (with the new Cargo.toml) will skip the compile and download the binary directly. cargo binstall --manifest-path ./Cargo.toml commitlint-rs

KeisukeYamashita commented 5 months ago

Let me update the scope of the PR as ci is not allowed 🙏