[FINISHED] Feature Freeze - Nov-Dec 2021 #6548
ZeWaka
announced in
Announcements
Replies: 3 comments
-
7 days notice. |
Beta Was this translation helpful? Give feedback.
0 replies
-
🥶☃️☃️🥶 |
Beta Was this translation helpful? Give feedback.
0 replies
-
The feature freeze is over. It has been successful and productive, though there's probably some changes I'd want to make for a future one. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hi all,
Starting on November 15th and ending ~Mid-December, we are initiating a Feature Freeze for Pull Requests submitted to the codebase.
Q: What is a feature freeze?
Essentially, we will not be merging any new PRs during the period that add new features as a general rule. Therefore, please do not ask any Developers to review or merge your feature PRs during this time.
Q: Why?
New features are the number one cause of bugs and instability in the codebase. The goal during this time is to fix a lot of these bugs.
We also want to take this time to do some larger overhauls that would normally break many people's contributions.
Edit: Additional note, this also applies to more than just code features, but things such as resprites, as these would interfere in reorganization projects.
Q: What if I submit a PR before the freeze but it isn't merged by the deadline?
That's fair game to be potentially merged during the freeze. This does not mean that we will merge them though.
PR merges are always up to Developer discretion.
Q: What about refactors?
While refactors are generally extremely good for code cleanliness, they can contribute instability to the codebase, especially when first done. Furthermore, they could conflict with other refactors that people are working on.
As always, it is suggested to ask a Developer first before working on/submitting a PR that refactors a game system.
Q: What about bugfixes?
Bugfixes will be merged as normal. Please link them to an associated issue filed on the GitHub.
However, this is not free reign to package new features and bugfixes in the same PR. We still want PRs to be properly split up into smaller PRs and not have parts combined.
Exceptions may be made if a small feature is added to fix a bug.
Beta Was this translation helpful? Give feedback.
All reactions