Skip to content

Conversation

@abhandage
Copy link
Contributor

@abhandage abhandage commented Jan 5, 2026

This pull request updates the Buildkite pipeline configuration to better separate CI workflows for the master branch from other branches, and introduces new scripts in package.json to support this separation. The changes improve clarity and reliability of CI runs, especially for the master branch, and remove the publish step from the pipeline.

Pipeline configuration updates:

  • Added new Buildkite steps specifically for the master branch: build_master and sauce_labs_master, each with their own commands and environment settings. These steps ensure that master runs use dedicated test scripts and Docker images. [1] [2]
  • Updated existing steps (build and sauce_labs) to exclude the master branch, so they only run on non-master branches. [1] [2]
  • Changed the pipeline's wait step to depend on the new master-specific steps, ensuring correct build sequencing.
  • Removed the :cloud: Publish step from the pipeline, so publishing no longer occurs automatically on master.

Script and package management updates:

  • Added new scripts to package.json (test-master, test-master:ci) for running tests on master without the --since master flag, and specified the Yarn version in the packageManager field.

@abhandage abhandage changed the title chore: specify yarn package manager version in package.json chore: (non-prod) Fix the pipeline to run all tests on master and remove publish step Jan 9, 2026
@abhandage abhandage changed the title chore: (non-prod) Fix the pipeline to run all tests on master and remove publish step (non-prod) Fix the pipeline to run all tests on master and remove publish step Jan 9, 2026
@abhandage abhandage changed the title (non-prod) Fix the pipeline to run all tests on master and remove publish step (non-prod) STRATCONN-6104: fix the pipeline to run all tests on master and remove publish step Jan 9, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request refactors the Buildkite CI pipeline to separate test execution workflows for the master branch from other branches, ensuring that all tests run on master without the --since master flag that limits testing to changed packages only. The publish step is also removed from the pipeline.

Key changes:

  • New test scripts (test-master, test-master:ci) that run all tests without the --since master flag
  • Separate Buildkite pipeline steps for master branch (build_master, sauce_labs_master) that use the new test scripts
  • Existing pipeline steps updated to exclude master branch using branches: "!master"

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 5 comments.

File Description
package.json Adds packageManager field for Yarn version pinning and introduces new test scripts for running all tests on master branch
.gitignore Adds .idea/ directory to ignored files for IDE configuration exclusion
.buildkite/pipeline.yml Creates separate CI steps for master branch, updates branch filters on existing steps, modifies wait dependencies, and removes publish step

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

abhandage and others added 2 commits January 9, 2026 15:54
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.

2 participants