Skip to content
This repository has been archived by the owner on Jan 11, 2019. It is now read-only.

Updated Url implementation #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cristianmedeiros
Copy link

I'm really new to rust, so please, be very critique about this PR. I know it's only a few lines but I want to learn and improve my code

What ?

As I started a new project with cargo, when running cargo install I got lots of errors

How ?

Removed the Url version constraint and updated the respective methods

@Valve
Copy link
Owner

Valve commented Jun 21, 2016

thanks! I'm not using this project and I thought nobody did. Are you using it as a playground or trying to really use it?

@cristianmedeiros
Copy link
Author

Both... I'm trying to use Rust in some side tasks on my job, I don't need to use it, but I'm "for fun and profit"

@Valve
Copy link
Owner

Valve commented Jun 21, 2016

OK, thanks! I am traveling at the moment but will look at this PR when I get back

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants