I started a rather big attempt at improving the code base on Flow towards avoiding unneeded
else statements and reducing indentation some time ago: https://github.com/neos/flow-development-collection/pull/443
This PR however touches so many places, that it is hard to merge without breaking all other PRs (and needs to be rebased pretty quickly if it isn’t merged).
I think the effort is very well worth it, as it improves code readability quite a bit, but the question is how to get it in with the least pain.
I guess there’s two ways to go about this, if it is to be attempted:
the quick shot, try to find a good time frame when no big PRs are outstanding, then rebase and merge it quickly and get all other PRs rebased
the slow poke, split the changes to many small focused PRs that only touch as few as possible files, then get those merged step by step
The first involves a one-time big effort and will cause a lot of side effects to other PRs at once. The second involves a long running effort and will only cause limited side effects to other PRs (but possibly repeatedly).
So, what’s your take on this?
- Don’t bother with this change, it’s not worth the effort
- Go for the single quick shot, do it asap
- Go for the single quick shot, but wait for the best point in time
- Go for the long running effort, do it step by step
If you choose “Go for the single quick shot, but wait for the best point in time”, please leave a comment what criteria you would put in place for “best point in time”.
Looking forward to your feedback