Skip to content

[build-tools] Require the sandbox worker token for sandbox jobs - #4808

Open
AHGIJMKLKKZNPJKQR wants to merge 1 commit into
tom/sandbox-worker-apifrom
tom/sandbox-worker-api-require
Open

AHGIJMKLKKZNPJKQR wants to merge 1 commit into
tom/sandbox-worker-apifrom
tom/sandbox-worker-api-require

Conversation

@AHGIJMKLKKZNPJKQR

@AHGIJMKLKKZNPJKQR AHGIJMKLKKZNPJKQR commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Why

Cleaning up old code from the old sandbox API.

How

removed LegacySandboxApi and its uses. This should only be merged once the new sandbox API is rolled out

Test Plan

tests pass

@codecov

codecov Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.65%. Comparing base (81edc06) to head (35e21df).

Additional details and impacted files
@@                    Coverage Diff                     @@
##           tom/sandbox-worker-api    #4808      +/-   ##
==========================================================
- Coverage                   76.67%   76.65%   -0.02%     
==========================================================
  Files                        1106     1105       -1     
  Lines                       55572    55512      -60     
  Branches                    12249    12272      +23     
==========================================================
- Hits                        42607    42549      -58     
+ Misses                      11976    11974       -2     
  Partials                      989      989              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AHGIJMKLKKZNPJKQR
AHGIJMKLKKZNPJKQR force-pushed the tom/sandbox-worker-api-require branch from 367fdd0 to 0ce96fa Compare November 3, 2026 16:00
@AHGIJMKLKKZNPJKQR
AHGIJMKLKKZNPJKQR force-pushed the tom/sandbox-worker-api-require branch from 0ce96fa to 4bc9c7a Compare November 4, 2026 11:23
@AHGIJMKLKKZNPJKQR
AHGIJMKLKKZNPJKQR force-pushed the tom/sandbox-worker-api-require branch 2 times, most recently from 9d712a2 to bc2003a Compare November 4, 2026 12:51
@AHGIJMKLKKZNPJKQR
AHGIJMKLKKZNPJKQR force-pushed the tom/sandbox-worker-api-require branch from bc2003a to 35e21df Compare November 4, 2026 13:02
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

✅ Thank you for adding a changeset!

@AHGIJMKLKKZNPJKQR
AHGIJMKLKKZNPJKQR marked this pull request as ready for review November 4, 2026 13:04
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.

1 participant