Skip to content

Conversation

slinkydeveloper
Copy link
Contributor

Part of #3184

Copy link
Contributor

@tillrohrmann tillrohrmann left a comment

Choose a reason for hiding this comment

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

Thanks for creating this PR @slinkydeveloper. I am doubtful about aggressively emitting warnings for every started invocation with the next version already. I think this is too little time for users to realistically upgrade their code to the latest SDK versions. Maybe we can do this once we get closer to the actual removal of the service protocols (something we haven't decided yet).

Copy link
Contributor

@tillrohrmann tillrohrmann left a comment

Choose a reason for hiding this comment

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

LGTM. +1 for merging.

@slinkydeveloper slinkydeveloper merged commit bec0f6a into restatedev:main Apr 28, 2025
25 checks passed
@slinkydeveloper slinkydeveloper deleted the issues/3184 branch April 28, 2025 13:31
@github-actions github-actions bot locked and limited conversation to collaborators Apr 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants