stevedlawrence commented on PR #1071:
URL: https://github.com/apache/daffodil/pull/1071#issuecomment-1693223916

   > although I don't know what to look for to check that the PR deletes 
everything needing to be removed.
   
   Yeah, I mostly did this through code inspection so it's hard to know for 
sure.
   
   I did find codcov a bit useful since it will show code that has lost 
coverage even if not changed in the PR. In this case we only had two tests for 
unions, so I'm less confident we had good coverage in the first place, but 
codecov did find one small function I had missed.


-- 
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