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

Remove RunnerState #705

Merged
merged 1 commit into from
Oct 28, 2023
Merged

Remove RunnerState #705

merged 1 commit into from
Oct 28, 2023

Conversation

Mark-Simulacrum
Copy link
Member

Store logs into a local stack variable that gets uploaded at the end. There's no reason to keep them in a semi-global place, and this makes it easier to capture more into the logs (e.g., should make us include retry logs automatically).

Store logs into a local stack variable that gets uploaded at the end.
There's no reason to keep them in a semi-global place, and this makes it
easier to capture more into the logs (e.g., should make us include retry
logs automatically).
@Mark-Simulacrum
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Oct 28, 2023

📌 Commit 5715bd3 has been approved by Mark-Simulacrum

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 28, 2023

⌛ Testing commit 5715bd3 with merge 0f43df9...

@bors
Copy link
Contributor

bors commented Oct 28, 2023

☀️ Test successful - checks-actions
Approved by: Mark-Simulacrum
Pushing 0f43df9 to master...

@bors bors merged commit 0f43df9 into rust-lang:master Oct 28, 2023
2 checks passed
@Mark-Simulacrum Mark-Simulacrum deleted the no-state branch October 28, 2023 23:17
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