Skip to content

add ./miri cargo for RA to invoke #2495

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Aug 20, 2022
Merged

add ./miri cargo for RA to invoke #2495

merged 1 commit into from
Aug 20, 2022

Conversation

RalfJung
Copy link
Member

RA expects a check command to check individual workspaces, whereas ./miri is designed to check/build/test all parts of Miri. So add a new ./miri cargo that performs just a single cargo invocation, but with the right env vars so that the build cache can be shared with ./miri check.

@RalfJung
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Aug 20, 2022

📌 Commit 0c3ad68 has been approved by RalfJung

It is now in the queue for this repository.

bors added a commit that referenced this pull request Aug 20, 2022
add ./miri cargo for RA to invoke

RA expects a check command to check *individual workspaces*, whereas `./miri` is designed to check/build/test all parts of Miri. So add a new `./miri cargo` that performs just a single cargo invocation, but with the right env vars so that the build cache can be shared with `./miri check`.
@bors
Copy link
Contributor

bors commented Aug 20, 2022

⌛ Testing commit 0c3ad68 with merge bdb5324...

@bors
Copy link
Contributor

bors commented Aug 20, 2022

💔 Test failed - checks-actions

@RalfJung
Copy link
Member Author

@bors rety

@RalfJung
Copy link
Member Author

@bors ping

@bors
Copy link
Contributor

bors commented Aug 20, 2022

😪 I'm awake I'm awake

@RalfJung
Copy link
Member Author

@bors r- r+

@bors
Copy link
Contributor

bors commented Aug 20, 2022

📌 Commit 0c3ad68 has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Aug 20, 2022

⌛ Testing commit 0c3ad68 with merge ab88e64...

@bors
Copy link
Contributor

bors commented Aug 20, 2022

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing ab88e64 to master...

@bors bors merged commit ab88e64 into rust-lang:master Aug 20, 2022
@RalfJung RalfJung deleted the ra branch August 20, 2022 14:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants