Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add warning about future deprecation of stepfunctions v1 provider #9198

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

Merged
merged 3 commits into from
Sep 26, 2023

Conversation

dominikschubert
Copy link
Member

Motivation

  • With 2.3 we want to start advertising the new v2 stepfunctions provider more broadly and also let users know that with the next major release, the defaults will change. Ideally anyone using stepfunctions should start to at least opt-in at some point and make sure their current applications are still working with v2.
  • With 3.0 the old (v1) provider will become deprecated and v2 will become the new default. Users can still opt-in to the old v1 provider.
  • With 4.0 the old (v1) provider will be removed completely.

Changes

  • Log a warning when the stepfunctions v1 provider is used. Only logged during initialization since a lot of people might not be affected at this point if they're not using the stepfunctions service at all.

Discussion

We should also make sure our language is somewhat similar between all the upcoming provider deprecations (s3, lambda, stepfunctions). /cc @bentsku @joe4dev

@dominikschubert dominikschubert added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label Sep 21, 2023
@dominikschubert dominikschubert self-assigned this Sep 21, 2023
@dominikschubert dominikschubert added this to the 2.3 milestone Sep 21, 2023
@github-actions
Copy link

github-actions bot commented Sep 21, 2023

LocalStack Community integration with Pro

       2 files         2 suites   1h 13m 39s ⏱️
2 220 tests 1 723 ✔️ 497 💤 0
2 221 runs  1 723 ✔️ 498 💤 0

Results for commit 4814885.

♻️ This comment has been updated with latest results.

@joe4dev joe4dev force-pushed the sfn-deprecation-warnings branch from 4814885 to f8fb908 Compare September 26, 2023 06:27
@joe4dev joe4dev merged commit 6287bfe into master Sep 26, 2023
@joe4dev joe4dev deleted the sfn-deprecation-warnings branch September 26, 2023 06:49
@coveralls
Copy link

Coverage Status

coverage: 83.056% (+0.002%) from 83.054% when pulling f8fb908 on sfn-deprecation-warnings into 9d020b1 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.

4 participants