-
Notifications
You must be signed in to change notification settings - Fork 8
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
Refactor putRecursive to avoid getRecursive and updating unchanged assets #32
Conversation
Signed-off-by: mikaellafs <[email protected]>
Signed-off-by: mikaellafs <[email protected]>
Signed-off-by: mikaellafs <[email protected]>
Signed-off-by: André Macedo <[email protected]>
Signed-off-by: André Macedo <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mikaellafs Everything worked fine in my tests here, but is there a reason to maintain the root
parameter in the putRecursive
method? I think there is no reason for it there after the refactor
Humm you're right, better we remove it |
Signed-off-by: mikaellafs <[email protected]>
4e9bc60
to
b7ef139
Compare
No description provided.