Skip to content

🐛 Build test apps outside Git repositories - #4962

Open
BenoitZugmeyer wants to merge 1 commit into
mainfrom
benoit/restore-test-app-files
Open

🐛 Build test apps outside Git repositories#4962
BenoitZugmeyer wants to merge 1 commit into
mainfrom
benoit/restore-test-app-files

Conversation

@BenoitZugmeyer

Copy link
Copy Markdown
Member

Motivation

Developers should be able to build test applications from source archives or Jujutsu workspaces without requiring Git metadata.

Changes

  • Preserve each test app’s manifest and lockfile while installing its optional local SDK packages.
  • Remove the build script’s dependency on Git commands.

Test instructions

From a source tree without Git metadata, run:

yarn build:apps --app vanilla

Verify that the app builds successfully and that its package.json and yarn.lock retain their original contents.

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.
  • Updated documentation and/or relevant AGENTS.md file

@BenoitZugmeyer
BenoitZugmeyer marked this pull request as ready for review August 18, 2026 13:47
@BenoitZugmeyer
BenoitZugmeyer requested a review from a team as a code owner August 18, 2026 13:47
@cit-pr-commenter-54b7da

Copy link
Copy Markdown

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 180.54 KiB 180.54 KiB 0 B 0.00%
Rum Profiler 8.43 KiB 8.43 KiB 0 B 0.00%
Rum Recorder 21.12 KiB 21.12 KiB 0 B 0.00%
Logs 57.04 KiB 57.04 KiB 0 B 0.00%
Rum Salesforce N/A 138.49 KiB N/A N/A N/A
Rum Slim 138.49 KiB 138.49 KiB 0 B 0.00%
Worker 22.96 KiB 22.96 KiB 0 B 0.00%
Rum Shopify N/A 200.11 KiB N/A N/A N/A
Rum-shopify Profiler N/A 8.43 KiB N/A N/A N/A
Rum-shopify Recorder N/A 3.72 KiB N/A N/A N/A

@datadog-prod-us1-3

datadog-prod-us1-3 Bot commented Aug 18, 2026

Copy link
Copy Markdown

Tests

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 77.21% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 99a618b | Docs | View more details | Give us feedback!

@sbarrio
sbarrio requested a review from thomas-lebeau August 19, 2026 07:06
@BenoitZugmeyer
BenoitZugmeyer force-pushed the benoit/restore-test-app-files branch from ee1df7a to 99a618b Compare August 19, 2026 12:11
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