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.

Sign up for GitHub

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

Jump to bottom

Fix warnings errors #648

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fix warnings errors #648

wants to merge 4 commits into from

Conversation

CraftSpider
Copy link
Collaborator

Supercedes #644

Bumps all dependencies to latest versions, and fixes the broken stacker cfg caught by the well-known cfgs lint.

@zesterer
Copy link
Owner

zesterer commented Jul 7, 2024

Thanks! Feel free to merge.

@CraftSpider
Copy link
Collaborator Author

How do you want to handle the MSRV stuff? It's test only, I can skip the upgrade for now if you'd like.

@zesterer
Copy link
Owner

zesterer commented Sep 1, 2024

Incidentally, I've just resolved this by removing test checks for MSRV

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zesterer zesterer zesterer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@CraftSpider @zesterer