Shopify / cli

Build apps, themes, and hydrogen storefronts for Shopify
https://shopify.dev
MIT License
436 stars 128 forks source link

Include Azure in `env_ci` categorisation #4814

Closed shauns closed 1 week ago

shauns commented 1 week ago

WHY are these changes introduced?

Adds support for detecting Azure CI environments in the CLI toolkit's continuous integration platform detection system.

WHAT is this pull request doing?

Adds Azure CI platform detection by checking for the presence of the TF_BUILD environment variable. When detected, the system will now correctly identify and return Azure-specific CI platform information.

How to test your changes?

  1. Set the TF_BUILD environment variable to "true"
  2. Run the CI platform detection code
  3. Verify that the system correctly identifies the environment as Azure CI
  4. Verify that the returned object contains:
    • isCI: true
    • name: 'azure'
    • appropriate metadata

Measuring impact

Checklist

shauns commented 1 week ago

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @shauns and the rest of your teammates on Graphite Graphite

github-actions[bot] commented 1 week ago

Coverage report

St.:grey_question:
Category Percentage Covered / Total
🟑 Statements
72.06% (+0.02% πŸ”Ό)
8356/11596
🟑 Branches
68.51% (+0.04% πŸ”Ό)
4042/5900
🟑 Functions 71.33% 2192/3073
🟑 Lines
72.44% (+0.02% πŸ”Ό)
7902/10908

Test suite run success

1893 tests passing in 867 suites.

Report generated by πŸ§ͺjest coverage report action from d95e1b3b76ca4b4c2d9b9a42aacc403ee3f2e27f

github-actions[bot] commented 1 week ago

We detected some changes at packages/*/src and there are no updates in the .changeset. If the changes are user-facing, run "pnpm changeset add" to track your changes and include them in the next release CHANGELOG.

shauns commented 1 week ago

Merge activity