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

[AN-226] Cromwell Google Batch API workspace setting #1503

Merged
merged 4 commits into from
Dec 10, 2024

Conversation

sam-schu
Copy link
Contributor

@sam-schu sam-schu commented Nov 26, 2024

Ticket: AN-226

Follow-up to broadinstitute/rawls#3141 to update the Rawls model version and the workspace settings Swagger documentation.

Testing:

  • Manually tested that the Swagger docs appear correctly.
  • Smoke tested various endpoints (for the Rawls model version bump).

Have you read CONTRIBUTING.md lately? If not, do that first.

I, the developer opening this PR, do solemnly pinky swear that:

  • I've followed the instructions if I've made any changes to the API, especially if they're breaking changes
  • I've updated the FISMA documentation if I've made any security-related changes, including auth, encryption, or auditing

In all cases:

  • Get two thumbsworth of review and PO signoff if necessary
  • Verify all tests go green
  • Squash and merge. Make sure your branch deletes; GitHub should do this for you.
  • Test this change deployed correctly and works on dev environment after deployment

@sam-schu sam-schu requested a review from a team as a code owner November 26, 2024 21:20
@sam-schu sam-schu requested review from calypsomatic and marctalbott and removed request for a team November 26, 2024 21:20
@sam-schu sam-schu marked this pull request as draft December 3, 2024 17:00
@sam-schu sam-schu changed the title [AN-225] Cromwell Google Batch API workspace setting Cromwell Google Batch API workspace setting Dec 3, 2024
@sam-schu sam-schu changed the title Cromwell Google Batch API workspace setting [AN-226] Cromwell Google Batch API workspace setting Dec 9, 2024
@sam-schu sam-schu marked this pull request as ready for review December 9, 2024 21:34
@sam-schu sam-schu merged commit a091f9c into develop Dec 10, 2024
13 checks passed
@sam-schu sam-schu deleted the sts_AN-225_batch_api_workspace_setting branch December 10, 2024 14:20
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.

4 participants