-
Notifications
You must be signed in to change notification settings - Fork 8
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
Remove SemVer #99
Remove SemVer #99
Conversation
@pjfanning Should I go ahead and merge this as its related to an issue you are talking about? |
I'm neutral on this. The change proposed in #90 is not massively impacting but I don't mind if we change the semver setting. |
I would like to see #90 done as it implied there is a legitimate reason why it was changed to |
@mdedetrich R2dbcExecutor is documented to be an internal class. For me, this means we can change its return types without too much fanfare. |
Maybe we should work out how big the changes in #101 will be before we decide if we need this. So far the public API changes are only in pekko-projection-r2dbc and not in the pekko-persistence-r2dbc module - maybe we could just remove semver marker from pekko-projection-r2dbc. |
Sure, do you want to go ahead with the PR to find out? |
I removed myself as a reviewer. That was uncool @mdedetrich. |
Apologies, this was an accident as there is another greg as part of committers. |
Due to #90