GutoVeronezi commented on PR #10140:
URL: https://github.com/apache/cloudstack/pull/10140#issuecomment-3048614823

   @abh1sar, your PR started as a feature to create a new instance from a VM 
backup, and now it has several changes not related to it. The PR has more than 
10k lines and misses a proper documentation/specification, which makes it 
difficult for anyone to review and understand what is going on, and make it 
more error prone. Please, separate the contexts in different PRs and add proper 
documentation/specification of workflows being changed.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to