-
Notifications
You must be signed in to change notification settings - Fork 18
Criteria/signoffs for merging PRs #52
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
Comments
Who would set this label? |
Should we also provide a template to write issues, and require people to maintain the issue description up to date with what has been possibly discussed, so that reviewers can easily refer to the corresponding issue of a PR to understand better the addressed problem, why it is important, etc.? |
If the bar is high for merging ("it represents the final version of future API, everyone agrees") then there will be minimal activity. It's difficult to see beneath the amazingly ambitious and prolific contributions by Scala Steward. Maybe what's needed is an artifact with a low bar for merging, Only efforts that survive that process would be promoted to |
Scala 3 provides a way to define “experimental” APIs. Those APIs don’t provide binary compatibility guarantees. I believe we should leverage that mechanism to ship APIs to the end-users without having to be 100% sure they are stable. However, we would still need a sort of “compat” library for cross-building (like |
honestly, I think there are several reasons this repo never went anywhere:
|
We could just rename it |
We now have several PRs that seem mergeable.
But I think we should probably hold off on actually merging any PRs here until we have some kind of signoff process. It probably doesn't need to be super formal, but this repo is still pretty new and I don't want standard library additions flying in under people's radar. Where “people” means: the Scala 2 team at Lightbend; the Dotty team at EPFL; key folks at the Scala Center such as @julienrf; and the community.
Off the top of my head: how about a label that indicates that something is considered mergeable? Then every so often, we can publicize that a batch of PRs is ready for hopefully-final review/signoff.
We use to have SLIP for this, but I don't suggest we try to bring that back. The basics I suggest we need are:
I think most of the time there will be consensus or near-consensus. If some particular change ends up being sufficiently controversial that it's not clear what to do, well, I guess we can cross that bridge when we come to it :-)
The text was updated successfully, but these errors were encountered: