Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Update dependencies, and fix warnings #40

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

Conversation

Dr-Emann
Copy link
Contributor

  • Make a cargo workspace at the top of the repo
  • Update all dependencies except combine to the newest version
    • cpal version 0.4 to 0.12 introduced some differences
  • Fix warnings

@yupferris
Copy link
Member

Hi! Thanks for going through and doing all of this, and thanks for being patient with my slow response time!

This looks great. My only concern is that LICENSE-THIRD-PARTY isn't updated, which is something that needs to be done when dependencies change.

I'd like to come up with a good way to do this; ideally it'd be automated, and perhaps this might help.

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.

None yet

2 participants