-
Notifications
You must be signed in to change notification settings - Fork 2.1k
createSourceEventStream: remove deprecated positional argument overload #3635
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
Conversation
✅ Deploy Preview for compassionate-pike-271cb3 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
Hi @yaacovCR, I'm @github-actions bot happy to help you with this PR 👋 Supported commandsPlease post this commands in separate comments and only one per comment:
|
IvanGoncharov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yaacovCR With exception of an issue with "positional test" it's ready to be merged.
7c212c0 to
3d49fda
Compare
See #3634, which introduced a named argument form for the exported function and deprecated the original positional argument overload. This is to be backported to v16.
This PR removes the deprecated format from v17. It is a BREAKING CHANGE and cannot be backported.
This PR depends on #3634.