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

Make it more obvious how to use shell with sandbox #16840

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

BretJohnson
Copy link
Member

@BretJohnson BretJohnson commented Aug 17, 2023

Description of Change

Add code + comment to make it more obvious how to switch the sandbox app to use shell.

@PureWeen and I chatted a bit about this.

@BretJohnson BretJohnson requested a review from a team as a code owner August 17, 2023 21:48
@Eilon Eilon added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Aug 17, 2023
@samhouts samhouts added the area-testing Unit tests, device tests label Aug 17, 2023
@rmarinho rmarinho merged commit 80a40e0 into main Aug 18, 2023
34 of 39 checks passed
@rmarinho rmarinho deleted the add-comment-shell-sandbox branch August 18, 2023 21:24
@github-actions github-actions bot locked and limited conversation to collaborators Dec 5, 2023
@samhouts samhouts added the fixed-in-8.0.0-rc.1.9171 Look for this fix in 8.0.0-rc.1.9171 label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions area-testing Unit tests, device tests fixed-in-8.0.0-rc.1.9171 Look for this fix in 8.0.0-rc.1.9171
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants