Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

Update code and packages #15

Merged
merged 1 commit into from
Aug 14, 2019

Conversation

AndrewRadev
Copy link
Contributor

Installing the package using cargo install cargo-open seems to fail for me on recent rust/cargo (rustc: 1.27.2, cargo: 1.27.0). Issue #14 describes the problem

Seems it's a matter of older package versions not compiling. This PR runs a cargo update and adjusts the code to work with them. I've taken some liberties with imports and naming, but I think I've mostly kept changes to a minimum. The error handling change I'm not sure about, I might update the PR if and when I figure out what the various errors and Nones mean in the cargo results :).

This gets cargo-open to work with a recent cargo version.
@carols10cents carols10cents merged commit 89df179 into carols10cents:master Aug 14, 2019
@carols10cents
Copy link
Owner

Thank you! Sorry for the long delay!

@AndrewRadev AndrewRadev deleted the update-packages branch August 14, 2019 17:42
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