Code review comment for lp:~danilo/linaro-android-frontend/bug-942671

Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

My only concern is that this text appears a bit misplaced - android-build.linaro.org is (to the best of my knowledge) is internal engineering system, to which we just provide open access, but not exactly a frontend into Linaro public resources. IMHO, such text would look better somewhere on releases.linaro.org or snapshots.linaro.org.

Another issue is that the text is long. Nobody else but our engineers would suffer thru constant scrolling to get to real build results.

Well, I assume that these organizational issues were considered already. Implementation-wise, looks good.

review: Approve

« Back to merge proposal