Re: What is status on PR for typeCalc and enums?

2019-05-16 Thread Beckerle, Mike
It is 11 commits, so needs a squash first. From: Sloane, Brandon Sent: Thursday, May 16, 2019 2:09:06 PM To: dev@daffodil.apache.org Subject: Re: What is status on PR for typeCalc and enums? I believe it is done. Steve's last comment is a +1, so I think we are

Re: What is status on PR for typeCalc and enums?

2019-05-16 Thread Sloane, Brandon
I believe it is done. Steve's last comment is a +1, so I think we are good there. I believe I addressed all of Josh's comments as well, but his review seems to have disappeared (likely due to a force-push on my end) From: Beckerle, Mike Sent: Thursday, May

What is status on PR for typeCalc and enums?

2019-05-16 Thread Beckerle, Mike
This PR has been sitting for a while with comments saying "not done reviewing". I approved it, though one such comment is mine. It has one approval, one not done by SL. Brandon, can you comment on whether you are ok with this being merged or have pending changes? This is an important