Skip to content

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