Skip to content

Extract CI test commands to shell scripts and refactor TLS features#427

Merged
yoshidan merged 3 commits intomainfrom
feature/fix_feature
Feb 22, 2026
Merged

Extract CI test commands to shell scripts and refactor TLS features#427
yoshidan merged 3 commits intomainfrom
feature/fix_feature

Conversation

@yoshidan
Copy link
Owner

Summary

  • Extract CI test commands to shell scripts under .github/scripts/
  • Refactor TLS feature flags across crates

CI Script Extraction

Background

Since we use pull_request_target, workflow files are loaded from the base branch (main).
This means changes to feature flags in fork PRs are not reflected in CI, causing test failures.

Changes

  • Add .github/scripts/ directory
  • Extract each test command to individual shell scripts
  • Update ci.yaml to call scripts instead of inline commands

Contributors can now modify scripts in their fork and have those changes reflected in CI.
Security is maintained through the existing safe to test label requirement.

TLS Feature Refactoring

Changes

  • Rename default-tls to rustls-tls as the default TLS backend
  • Add native-tls feature to support native TLS backend
  • Apply to: foundation/auth, pubsub, spanner, storage, kms, bigquery, artifact-registry

@yoshidan yoshidan added the safe to test safe to test label Feb 22, 2026
@yoshidan yoshidan merged commit 4efdeb1 into main Feb 22, 2026
0 of 9 checks passed
@yoshidan yoshidan deleted the feature/fix_feature branch February 22, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test safe to test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant