Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: standalone testing check #206

Merged
merged 30 commits into from
Feb 7, 2025
Merged

Conversation

ryan-timothy-albert
Copy link
Contributor

@ryan-timothy-albert ryan-timothy-albert commented Feb 5, 2025

Moves forward with us running testing by default as a PR check

  • Allows the user to attach their own ACTION_PAT to automatically trigger tests
  • Also leaves the door open to use using our app to trigger tests either with empty commits or opening/closing PRs

@ryan-timothy-albert ryan-timothy-albert requested a review from a team as a code owner February 5, 2025 21:20
@ryan-timothy-albert
Copy link
Contributor Author

ryan-timothy-albert commented Feb 5, 2025

This actual auto generated github file would look something like the following [autogenerated of course]

name: Test Petstore
permissions:
  checks: write
  contents: write
  pull-requests: write
  statuses: write

on:
  workflow_dispatch:
    inputs:
      target:
        description: Specific target to test
        type: string
        required: true
  pull_request:
    paths:
      - 'sdk/**'
    branches:
      - main
  push: # if you have setup your own ACTION_PAT feel free to remove this trigger
      branches:
        - speakeasy-sdk-regen-*
jobs:
  test:
    uses: speakeasy-api/sdk-generation-action/.github/workflows/sdk-test.yaml@v15
    with:
      target: ${{ github.event.inputs.target }}
    secrets:
      github_access_token: ${{ secrets.GITHUB_TOKEN }}
      speakeasy_api_key: ${{ secrets.SPEAKEASY_API_KEY }}

action_pat: ${{ secrets.ACTION_PAT }}

Optionally added to generation action

@TristanSpeakEasy
Copy link
Member

LGTM but I will let @chase-crumbaugh approve

@ryan-timothy-albert ryan-timothy-albert merged commit 8cc4d70 into main Feb 7, 2025
2 checks passed
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.

3 participants