stevedlawrence commented on PR #253:
URL: https://github.com/apache/daffodil-vscode/pull/253#issuecomment-1213388488

   Please squash into a single commit and rebase on top of the main branch. And 
please make the comment desribe what and why you're adding. It looks like some 
files are just copied form Daffodil source or other pleases. Please mention 
that and if any changes were made so we know what and what not to focus on.
   
   Also, in the future, please try to split of PR's into smaller bite size 
chunks. This PR closes 8 issues. I would usually prefer 8 separate PR's 
specific to each issue. That makes things much easier to review.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to