Closed nitingujralBCH closed 6 years ago
I second @heathharrelson 's request. There are a number of minor things that should be addressed before merging but they should be quick to resolve. Thanks.
@nitingujralBCH are you able to make the changes suggested in this PR? We are happy to merge it once they are made.
It looks like all of my concerns have been addressed by the latest batch of commits. I will merge this and get out a PR that fixes broken unit tests today.