Skip to content

Run SDK tests on push to main branch#390

Merged
SteveSandersonMS merged 1 commit intomainfrom
stevesa/ci-run-tests-on-main
Feb 6, 2026
Merged

Run SDK tests on push to main branch#390
SteveSandersonMS merged 1 commit intomainfrom
stevesa/ci-run-tests-on-main

Conversation

@SteveSandersonMS
Copy link
Contributor

All SDK test workflows were only running on pull requests, not on merges to main. This adds \push\ triggers for the \main\ branch so tests run after PRs are merged.

@SteveSandersonMS SteveSandersonMS requested a review from a team as a code owner February 6, 2026 09:58
Copilot AI review requested due to automatic review settings February 6, 2026 09:58
@SteveSandersonMS SteveSandersonMS force-pushed the stevesa/ci-run-tests-on-main branch from e4e3f9b to d406d90 Compare February 6, 2026 09:58
@SteveSandersonMS SteveSandersonMS merged commit 22fb64f into main Feb 6, 2026
21 checks passed
@SteveSandersonMS SteveSandersonMS deleted the stevesa/ci-run-tests-on-main branch February 6, 2026 09:59
Copy link
Contributor

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 PR adds push triggers to all four SDK test workflows (Python, Node.js, Go, and .NET) to ensure tests run automatically after PRs are merged to the main branch. Previously, these workflows only ran on pull requests, meaning the main branch didn't get continuous testing after merges.

Changes:

  • Added push triggers for the main branch to Python, Node.js, Go, and .NET SDK test workflows
  • Configured path filters for push triggers to match relevant SDK and test directories

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
.github/workflows/python-sdk-tests.yml Added push trigger for main branch with path filters for Python SDK
.github/workflows/nodejs-sdk-tests.yml Added push trigger for main branch with path filters for Node.js SDK
.github/workflows/go-sdk-tests.yml Added push trigger for main branch with path filters for Go SDK
.github/workflows/dotnet-sdk-tests.yml Added push trigger for main branch with path filters for .NET SDK
Comments suppressed due to low confidence (3)

.github/workflows/python-sdk-tests.yml:17

  • The path filters for the push trigger are incomplete compared to the pull_request trigger. The pull_request trigger includes additional exclusions for .editorconfig and image files (png, jpg, jpeg, gif, svg) that are missing from the push trigger. These path filters should be identical to ensure consistent behavior whether tests run on a pull request or after merging to main.
  pull_request:
    paths:
      - 'python/**'
      - 'test/**'
      - 'nodejs/package.json'
      - '.github/workflows/python-sdk-tests.yml'
      - '!**/*.md'
      - '!**/LICENSE*'

.github/workflows/nodejs-sdk-tests.yml:17

  • The path filters for the push trigger are incomplete compared to the pull_request trigger. The pull_request trigger includes additional exclusions for image files (png, jpg, jpeg, gif, svg) that are missing from the push trigger. These path filters should be identical to ensure consistent behavior whether tests run on a pull request or after merging to main.
  pull_request:
    paths:
      - 'nodejs/**'
      - 'test/**'
      - '.github/workflows/nodejs-sdk-tests.yml'
      - '!**/*.md'
      - '!**/LICENSE*'
      - '!**/.gitignore'

.github/workflows/dotnet-sdk-tests.yml:17

  • The path filters for the push trigger are incomplete compared to the pull_request trigger. The pull_request trigger includes additional exclusions for jpeg, gif, and svg image files that are missing from the push trigger. These path filters should be identical to ensure consistent behavior whether tests run on a pull request or after merging to main.
  pull_request:
    paths:
      - 'dotnet/**'
      - 'test/**'
      - 'nodejs/package.json'
      - '.github/workflows/dotnet-sdk-tests.yml'
      - '!**/*.md'
      - '!**/LICENSE*'
      - '!**/.gitignore'
      - '!**/.editorconfig'
      - '!**/*.png'

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