-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Refactors dav app commands. #39166
Refactors dav app commands. #39166
Conversation
Hey @szaimen, sorry to bother you. |
443c32e
to
1ea5d43
Compare
Conflicts resolved. |
Hey @icewind1991, Saw the retriggered failed workflows and I wanted to let you know It's been more than a month now that not a single pull request of mine has been able to pass the Cypress workflows. |
I will rebase the branch - if you want to do some additional work, you will need to pull first @fsamapoor |
1ea5d43
to
5b6e419
Compare
To improve code readability. Signed-off-by: Faraz Samapoor <fsa@adlas.at>
5b6e419
to
d13874f
Compare
No additional work is needed. I just rebased it again to pass the failed DCO workflow. |
Summary
I have made some adjustments to the
apps/dav/lib/Command
classes to improve the code readability.The improvements in this PR include but are not limited to:
Checklist