Skip to content
This repository was archived by the owner on Feb 18, 2025. It is now read-only.

Bump msrv and unpin cfg-if. #32

Closed
wants to merge 1 commit into from

Conversation

dvc94ch
Copy link

@dvc94ch dvc94ch commented Jun 24, 2020

No description provided.

@soc
Copy link
Collaborator

soc commented Jun 24, 2020

What's the intention of this?

@dvc94ch
Copy link
Author

dvc94ch commented Jun 24, 2020

It causes a failure to resolve the dependency if there is another dependency in the graph is using a higher version. Not sure why cargo doesn't include it twice in the dependency graph.

@dvc94ch
Copy link
Author

dvc94ch commented Jun 25, 2020

So? Do you want me to create a reproducible test case? It's not like I waste my time with PR's to working code

@soc
Copy link
Collaborator

soc commented Jun 26, 2020

No, it's fine. I'll probably go the #26 route and drop cfg-if instead.

@dvc94ch dvc94ch mentioned this pull request Jun 26, 2020
@dvc94ch dvc94ch closed this Jun 26, 2020
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