Skip to content

Conversation

@tuliocasagrande
Copy link
Contributor

Description of changes:

This PR fixes the documentation for the definition parameter which expects a State or Chain and not a string.

Also add Args: to update() to correctly render in the doc page: https://aws-step-functions-data-science-sdk.readthedocs.io/en/latest/workflow.html#stepfunctions.workflow.Workflow.update.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@StepFunctions-Bot
Copy link
Contributor

AWS CodeBuild CI Report

  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@StepFunctions-Bot
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: StepFunctionsPythonSDK-integtests
  • Commit ID: fe6e178
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@vaib-amz
Copy link
Contributor

vaib-amz commented May 7, 2020

Hey, thanks for the fix!

Vaib

@shunjd shunjd merged commit bb06bfb into aws:master May 7, 2020
@shunjd
Copy link
Contributor

shunjd commented May 7, 2020

Thanks a lot for the fix. @tuliocasagrande

@tuliocasagrande tuliocasagrande deleted the fix-definition-param branch May 7, 2020 21:12
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