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

chore: minor fixes #158

Merged
merged 3 commits into from
Jul 11, 2023
Merged

Conversation

xxchan
Copy link
Contributor

@xxchan xxchan commented Jul 8, 2023

No description provided.

warning: some crates are on edition 2021 which defaults to `resolver = "2"`, but virtual workspaces default to `resolver = "1"`
note: to keep the current resolver, specify `workspace.resolver = "1"` in the workspace root's manifest
note: to use the edition 2021 resolver, specify `workspace.resolver = "2"` in the workspace root's manifest

As a library, this isn't important as end-user application decides the resolver finally. Just remove the warning.
failed to compile on latest nightly due to value-bag
@xxchan
Copy link
Contributor Author

xxchan commented Jul 11, 2023

🤔 It seems resolver has some impacts.

@xxchan
Copy link
Contributor Author

xxchan commented Jul 11, 2023

I think it's because for resolver=1, 'cfg(madsim)' and 'cfg(not(madsim))' are unified.

@wangrunji0408 wangrunji0408 merged commit 11e078a into madsim-rs:main Jul 11, 2023
12 checks passed
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