Skip to content

Test: Verify build optimization test-skipping#7208

Closed
Groenbech96 wants to merge 1 commit intofeature/build-optimization-app-level-filteringfrom
test/build-optimization-verify
Closed

Test: Verify build optimization test-skipping#7208
Groenbech96 wants to merge 1 commit intofeature/build-optimization-app-level-filteringfrom
test/build-optimization-verify

Conversation

@Groenbech96
Copy link
Contributor

Summary

  • Trivial Shopify doc comment change to verify build optimization works
  • Based on the feature branch so the only diff is the Shopify file change
  • Build optimization should skip tests for all non-Shopify test apps
  • This PR is for testing only — do not merge

What to look for in the logs

  • BUILD OPTIMIZATION: Comparing <sha>...<sha> — confirms event payload SHAs are extracted
  • BUILD OPTIMIZATION: Changed files (N): — confirms changed files are detected
  • BUILD OPTIMIZATION: SKIPPING tests for '<app>' — confirms non-Shopify apps are skipped
  • BUILD OPTIMIZATION: RUNNING tests for 'Shopify Connector-Test' — confirms Shopify tests still run

Context

Tests PR #7111 fix for shallow clone git diff failure in Get-ChangedFilesForCI.

🤖 Generated with Claude Code

@Groenbech96 Groenbech96 requested a review from a team as a code owner March 18, 2026 09:05
@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Mar 18, 2026
This commit adds a doc comment to a Shopify enum to create a PR where
the only app change is in Shopify. The build optimization should skip
tests for all non-Shopify apps.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Groenbech96 Groenbech96 force-pushed the test/build-optimization-verify branch from d4605a9 to 1e63f95 Compare March 18, 2026 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant