Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

[SFN] Support for DescribeStateMachineForExecution#8954

Merged
MEPalma merged 4 commits into
masterfrom
MEP-sfn-DescribeStateMachineForExecution
Aug 25, 2023
Merged

[SFN] Support for DescribeStateMachineForExecution#8954
MEPalma merged 4 commits into
masterfrom
MEP-sfn-DescribeStateMachineForExecution

Conversation

@MEPalma
Copy link
Copy Markdown
Contributor

@MEPalma MEPalma commented Aug 22, 2023

Motivation

Adds support for missing api action DescribeStateMachineForExecution.

Changes

Added support in the provider for v2.

@MEPalma MEPalma added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label Aug 22, 2023
@MEPalma MEPalma added this to the 2.3 milestone Aug 22, 2023
@MEPalma MEPalma self-assigned this Aug 22, 2023
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 80.836% (-0.003%) from 80.839% when pulling 8f23bf4 on MEP-sfn-DescribeStateMachineForExecution into ed5f065 on master.

Copy link
Copy Markdown
Member

@dominikschubert dominikschubert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Please adjust the markers before merging though. Since you've been collecting snapshots it would seem they should be marked as validated

Comment thread tests/aws/services/stepfunctions/v2/test_sfn_api.py Outdated
@github-actions
Copy link
Copy Markdown

LocalStack Community integration with Pro

       2 files         2 suites   1h 29m 50s ⏱️
2 131 tests 1 668 ✔️ 463 💤 0
2 132 runs  1 668 ✔️ 464 💤 0

Results for commit 12ecc9b.

@MEPalma MEPalma merged commit 26b957a into master Aug 25, 2023
@MEPalma MEPalma deleted the MEP-sfn-DescribeStateMachineForExecution branch August 25, 2023 09:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants