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

Update vcr cassettes #121

Merged
merged 1 commit into from
Feb 27, 2025
Merged

Update vcr cassettes #121

merged 1 commit into from
Feb 27, 2025

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Feb 26, 2025

Updated vcr cassettes

Via event: workflow_dispatch

  • If "workflow_dispatch" it was triggered by a person clicking the "Run workflow" button in the GitHub UI Actions tab
  • If "schedule" it was triggered by a scheduled event (see the .github/workflows/api-rewrite-cassettes.yml file for the schedule)

Review, then either merge or close this PR

  • A cursory review is sufficient (e.g., just look at a few cassette *yml files; though if anything is concerning, please investigate and discuss with Scott or Sean)

What is this?

  • This is a pull request that updates the vcr cassettes for the Cromwell API tests. Read more about how tests work in the README. We use the vcrpy Python library to record and replay HTTP requests and responses (called "cassettes"). These cassettes need to be re-recorded every so often to ensure that tests reflect the reality of the PROOF and Cromwell APIs.

Note that workflows are not run for this PR because actions cannot trigger other workflows

@github-actions github-actions bot force-pushed the create-pull-request/rewrite-cassettes branch from 4a7c413 to 89dca5a Compare February 26, 2025 22:32
@seankross
Copy link
Collaborator

This looks great, thank you!

@sckott sckott merged commit 4204a9b into main Feb 27, 2025
@sckott sckott deleted the create-pull-request/rewrite-cassettes branch February 27, 2025 16:56
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