Skip to content

chore: Update contract-tests action to v1.3.0 and pass token to SSE step#264

Merged
kinyoklion merged 1 commit into
mainfrom
rlamb/sdk-2323/update-contract-tests-action
May 5, 2026
Merged

chore: Update contract-tests action to v1.3.0 and pass token to SSE step#264
kinyoklion merged 1 commit into
mainfrom
rlamb/sdk-2323/update-contract-tests-action

Conversation

@kinyoklion
Copy link
Copy Markdown
Member

@kinyoklion kinyoklion commented May 4, 2026

Summary

  • Bumps launchdarkly/gh-actions/actions/contract-tests from contract-tests-v1.0.1 to contract-tests-v1.3.0 for the SDK contract tests step.
  • Replaces the inline curl ... | sh SSE downloader invocation with the same contract-tests@contract-tests-v1.3.0 action (repo: sse-contract-tests, branch: v2.0.0, version: v2, debug_logging: true, enable_persistence_tests: false) so the existing harness version is preserved.
  • Passes token: ${{ inputs.github_token }} to the SSE step so the downloader script and harness fetch are no longer subject to anonymous GitHub API rate limits.
  • Adds enable_persistence_tests: false to the SDK contract tests step to preserve the previous behavior, since v1.2.0+ flips the default to true.

Note

Medium Risk
Moderate CI risk: updates external GitHub Action versions and changes how SSE contract tests are invoked, which could alter test execution or fail due to action/harness behavior changes.

Overview
Updates the shared CI composite to run contract tests via launchdarkly/gh-actions/actions/contract-tests@contract-tests-v1.3.0 (from v1.0.1) and explicitly disables persistence tests to preserve prior behavior.

Replaces the inline SSE harness curl | sh invocation with the same contract-tests action (pinning sse-contract-tests v2.0.0/v2) and passes token: ${{ inputs.github_token }} to avoid anonymous GitHub API rate limits.

Reviewed by Cursor Bugbot for commit f83277b. Bugbot is set up for automated code reviews on this repo. Configure here.

Bumps the SDK contract-tests action from v1.0.1 to v1.3.0 and replaces
the inline SSE downloader curl with the same action, so the GitHub token
is passed through to both steps and avoids rate limiting on the
downloader script.
@kinyoklion kinyoklion marked this pull request as ready for review May 5, 2026 15:53
@kinyoklion kinyoklion requested a review from a team as a code owner May 5, 2026 15:53
@kinyoklion kinyoklion merged commit ad4d583 into main May 5, 2026
9 checks passed
@kinyoklion kinyoklion deleted the rlamb/sdk-2323/update-contract-tests-action branch May 5, 2026 17:28
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