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

ISSUE-16 - Remove use of cargo::core: #147

Merged

Conversation

jmcconnell26
Copy link
Contributor

  • Replace usage of cargo core dependency kind
  • Remove use of package set

Signed-off-by: joshmc [email protected]

@jmcconnell26 jmcconnell26 force-pushed the ISSUE-16-RemoveUseOfPackageSet branch from 3044f48 to b3c6000 Compare November 27, 2020 19:41
* Replace usage of cargo core dependency kind
* Remove use of package set

Signed-off-by: joshmc <[email protected]>
@jmcconnell26 jmcconnell26 force-pushed the ISSUE-16-RemoveUseOfPackageSet branch from b3c6000 to ccbc249 Compare November 27, 2020 19:43
.unwrap()
.push(dependency.clone());
}

dependency_type_nodes
}

// cargo_metadata::DependencyKind doesn't implement Eq or Hash, and so can't
Copy link
Contributor

Choose a reason for hiding this comment

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

Make an upstream PR or issue, if you have the time.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good! Have landed oli-obk/cargo_metadata#148 to resolve

@anderejd anderejd merged commit 94ae1c7 into geiger-rs:master Nov 29, 2020
@jmcconnell26 jmcconnell26 deleted the ISSUE-16-RemoveUseOfPackageSet branch May 10, 2022 18:15
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.

2 participants