You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ariatosca.apache.org by Thomas Nadeau <tn...@gmail.com> on 2018/02/08 14:39:35 UTC

code review for ARIA-118

DJ and I sat down this morning and did a pairwise code review of his patch
set.
The functionally things look cool.  My requests for changes have to do with
adding more comments and transforming literal strings/integers into
constants which have been discussed in detail during our code review. So he
is going to withdraw his PR and amend things a bit and resubmit.

--Tom