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

Use cargo_metadata in cargo-miri #2450

Merged
merged 2 commits into from
Jul 29, 2022
Merged

Conversation

avrong
Copy link
Contributor

@avrong avrong commented Jul 29, 2022

Closes #2393

Added cargo_metadata to cargo-miri and changed metadata from manual parsing to cargo_metadata invocations. Thus, removed local Metadata struct too.

Happy to fix if anything isn't right :)

@oli-obk
Copy link
Contributor

oli-obk commented Jul 29, 2022

You'll need to run ./miri fmt && ./miri clippy -Dwarnings to pass CI

@oli-obk
Copy link
Contributor

oli-obk commented Jul 29, 2022

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 29, 2022

📌 Commit 01a6109 has been approved by oli-obk

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jul 29, 2022

⌛ Testing commit 01a6109 with merge 44dc49f...

@bors
Copy link
Collaborator

bors commented Jul 29, 2022

☀️ Test successful - checks-actions
Approved by: oli-obk
Pushing 44dc49f to master...

@bors bors merged commit 44dc49f into rust-lang:master Jul 29, 2022
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.

Use cargo_metadata in cargo-miri
3 participants