Code review comment for lp:~canonical-ci-engineering/uci-engine/nfss

Revision history for this message
Andy Doan (doanac) wrote :

I have some comments and some notes to help explain things in case other people review this.

I have no business trying to review the j/s angular stuff.

My only other concern is the directory, nf-stats-service/web_static/v. I'd prefer to merge this without carrying that along. I'd give bonus points if our merge history didn't include it, but that might be too painful to deal with. VCS is forever, so each little 500k mishap in an MP can start to add up over time.

« Back to merge proposal