racv/openwrap — comment on commit 24eaf7bb3c095445c5012ccbdd181f81ce366c5c
File:  line 0
By serialseb:

Happy to take the pull request, couple of comments:

- can the tests in /src/OpenWrap.Tests/Build/build_instruction_emitter, especially around the addition of contract assemblies?
- can we not import System.IO it makes reading code based on the OFS.IO.Path a bit difficult to read (bad choice to have conflicting type names, will be resolved in OFS 2.0 but we won't be taking that in until OW 2.1 or beyond)
- in general don't change the version file when branching off master, messes with the build process :) For this one not an issue as the next release will be 2.0.3 but in the future :)
- see inline for other comment :)