-
Notifications
You must be signed in to change notification settings - Fork 53
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
3.x ? #1135
Comments
First, let us collect those relevant issues to provide an overview. I first created a label, but a milestone is more appropriate. I will call the milestone "Version 3.0". |
I already added 3 issues to the created milestone. Anything else?
I think it depends on how many features we plan to add. From looking at other distributed task libraries, there is not much missing
Yes, where it makes sense. If we replace something with something better, but leave the old thing there.
If we decide to do a beta version then yes, I think it's worth having a separate branch. There are currently only 3 issues, and it seems to me that we can tackle them quite fast. So, I am not sure if it is worth it. On the other hand, the worker refactoring seems to be quite large, and having a beta version would give us some safety (@onlyann, what do you think?).
I am absolutely in favor of this. I wanted to clean up the comments there all the time anyway.
And maybe some posting on Hacker News 😜. |
I fully support the introduction of a beta to obtain feedback and iterate over changes, if that is not too much work to put in place. Good idea to introduce a tag. I will take the opportunity to open new issues about some breaking change suggestions. |
Should we create a |
Sure! Though I'm not sure we need to change things in the CI (we can add the fact that the CI runs on the branch after we merge), but we can apply the same branch protections rules. |
Hm. If I understand it correctly, we want all the breaking stuff for the 3.0 release in that branch. So, there will be multiple PRs against this branch. Don't we want to run CI against those PRs before they get merged?
Will you set it up? I don't think that I have the permissions to change the protection rules. |
Unless I'm mistaken, CI runs on all PRs regardless of the target branch AND on main to double-check that what we merged didn't cause an issue. If we target
Yep. |
Here you are. There's a |
Relates to #1131 and #1114 (and a few others)
I was thinking about whether we were doing breaking changes in #1114 and you mentionned it in #1131 so maybe it's break time.
Version numbers are free. The downside of doing a major release is that we're using our limited voice power to call the attention of every downstream user of the lib, and if we do it too often, there's fatigue and "boy who cried wolf" effect. We already did 2 major releases (although in quick succession) at the beginning of the year, I think it's perfectly OK to do a new one especially since we're adding plenty of good stuff to the lib (thank you both !), but it might be worth it to try and think about the decisions we did that we ended up changing, and see if we identify a pattern in what led to that so as not to find ourselves in the same situation again too soon. To be clear: I'm not criticizing any decision we made, but trying to get as many opportunities as we can to improve ourselves, our vision for the lib, the processes etc.
If we want to drop sync_to_async at the same time, it might be an opportunity.
Things to note:
3.x
or we can develop3.x
onmain
. Depends whether we think it's going to be long and whether we think we might want to do things in parallel.(also, each time we do a major release, it seems we're featured in Postgres Weekly, but I don't think they'll do it if we release too often so let's not abuse this :D )
ping @onlyann @medihack
The text was updated successfully, but these errors were encountered: