Yes, I fully support this. Let’s do this. Renat Akhmerov @ Mirantis Inc. > On 13 Aug 2015, at 17:44, ELISHA, Moshe (Moshe) <moshe.elisha at alcatel-lucent.com> wrote: > > Hey, > > Following our discussion in the IRC, when pushing UI changes please upload a screenshot of the result and add a link in the commit message. > This will allow us to better understand the change and will also allow non-UI developers to comment on the change. > Having the screenshot link in the commit message will allow the developer to update the screenshot if there are visible changes as a result of the reviews. > > If the UI change is very minor or it is only infra and there are no visible changes – use “Screenshot: N/A”. > > You can see an example at the recent “Task details overview screen” push [1] > > [1] https://review.openstack.org/#/c/212489/ <https://review.openstack.org/#/c/212489/>__________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: OpenStack-dev-request at lists.openstack.org <mailto:OpenStack-dev-request at lists.openstack.org>?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150817/d922436b/attachment.html>