Code review comment for lp:~aacid/unity8/removeUnusedHasAttributes

Revision history for this message
Ying-Chun Liu (paulliu) wrote :

Looks good. Working good.
BTW, why model is moved after numOfAttributes?. Doesn't matter, just curious.

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Yes.

 * Did CI run pass? If not, please explain why.
Yes previously.

review: Approve

« Back to merge proposal