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

add deprecation comments #650

Merged
merged 1 commit into from
Nov 20, 2024
Merged

Conversation

famarting
Copy link
Contributor

Description

adds comments for #634

Issue reference

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Signed-off-by: Fabian Martinez <[email protected]>
@famarting famarting requested review from a team as code owners November 20, 2024 15:05
Copy link

codecov bot commented Nov 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.37%. Comparing base (6c59092) to head (cdb0592).
Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #650      +/-   ##
==========================================
- Coverage   62.52%   62.37%   -0.15%     
==========================================
  Files          56       56              
  Lines        4139     4205      +66     
==========================================
+ Hits         2588     2623      +35     
- Misses       1425     1454      +29     
- Partials      126      128       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@@ -230,24 +230,38 @@ type Client interface {
ImplActorClientStub(actorClientStub actor.Client, opt ...config.Option)

// StartWorkflowBeta1 starts a workflow.
// Deprecated: Please use the workflow client (github.com/dapr/go-sdk/workflow).
// These methods for managing workflows are no longer supported and will be removed in the 1.16 release.
Copy link
Contributor

Choose a reason for hiding this comment

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

1.16 or 1.17?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

IDK

Copy link
Member

Choose a reason for hiding this comment

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

1.16

@yaron2 yaron2 merged commit 87659bf into dapr:main Nov 20, 2024
17 of 18 checks passed
@famarting famarting deleted the add-deprecation-comments branch November 21, 2024 14:56
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.

3 participants