-
Notifications
You must be signed in to change notification settings - Fork 20
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
Reduce redundant else-branching (after throws), add missing @throws annotations #138
Conversation
…nnotations Cherry-picked Shardj/zf1-future@5e8bb2e Cherry-picked Shardj/zf1-future@5997d40 Co-authored-by: Elan Ruusamäe <glen@pld-linux.org>
0ee0fc9
to
fe76c17
Compare
php 8.0 job needs retry. i don't have permission to do so myself. some temporary error setting up container. |
@glensc But do we really need all the code beautifying? let the zf1 be the ugly beast, only taming it to make it work without errors on newer php versions. I don't want to follow the path of others trying to make the codebase look cleaner or more modern :) |
or is it a matter of a IDE complaining there, for redundant |
that's the thing, we don't need, but doing so makes it easier to pull changes from the other repo if the codebase is not that diverged. i reviewed the diffs here many times, these seem to do what they describe. no extra code, no side effects. |
I already excluded whitespace and formatting changes (if you open the original pull request), which makes merging the codebase more difficult again, but at least |
Extracted annotations: so you can decide about the other changes here later. |
Oh that's lovely, thanks @glensc . I will look into it |
decided to go with #140 instead |
Carry Shardj/zf1-future#104.
cc @kstenschke