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

Cargo script mvp #17110

Merged
merged 4 commits into from
Apr 19, 2024
Merged

Cargo script mvp #17110

merged 4 commits into from
Apr 19, 2024

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Apr 19, 2024

Based on #15456,

As the original PR stated, detached files are still horrendous to work with.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 19, 2024
@Veykril Veykril marked this pull request as draft April 19, 2024 12:56
@Veykril Veykril force-pushed the cargo-script-mvp branch 4 times, most recently from 2c5beeb to 24a84ce Compare April 19, 2024 16:10
@bors
Copy link
Contributor

bors commented Apr 19, 2024

☔ The latest upstream changes (presumably #17108) made this pull request unmergeable. Please resolve the merge conflicts.

@Veykril Veykril marked this pull request as ready for review April 19, 2024 16:26
@Veykril
Copy link
Member Author

Veykril commented Apr 19, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Apr 19, 2024

📌 Commit 0ce7179 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Apr 19, 2024

⌛ Testing commit 0ce7179 with merge c83d8cf...

@bors
Copy link
Contributor

bors commented Apr 19, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing c83d8cf to master...

@bors bors merged commit c83d8cf into rust-lang:master Apr 19, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants