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

Fix full_crate() #11

Closed
wants to merge 2 commits into from
Closed

Fix full_crate() #11

wants to merge 2 commits into from

Conversation

joseluis
Copy link
Contributor

@joseluis joseluis commented Apr 6, 2019

Fixes #7

Removed unexistant VersionDownloads.id field

e.g. https://crates.io/api/v1/crates/hyper-native-tls/downloads

This is a working example. Save as src/bin/fix-fullcrate.rs and run cargo run --bin fix-fullcrate to check:

extern crate crates_io_api;
use crates_io_api::SyncClient;

fn main() {
    let client = SyncClient::new();
    let crate_name = "hyper-native-tls";
    let c = client.full_crate(crate_name, false).unwrap();
    println!("name={}", c.name);
}

@theduke
Copy link
Owner

theduke commented Apr 9, 2019

Thanks.

I pushed your commit to master myself since this PR includes a commit from the other one and I couldn't force push to your branch.

@theduke theduke closed this Apr 9, 2019
@joseluis joseluis deleted the fix-fullcrate branch April 9, 2019 13:30
@joseluis
Copy link
Contributor Author

joseluis commented Apr 9, 2019

Right, I missed that. It's all good

@theduke
Copy link
Owner

theduke commented Apr 9, 2019

I also already released a 0.4.1 version with the fix, FYI.

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.

fn full_crate() doesn't work
2 participants