Yes, I fully support this. Let’s do this. Renat Akhmerov @ Mirantis Inc.
> On 13 Aug 2015, at 17:44, ELISHA, Moshe (Moshe) > <[email protected]> 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: [email protected] > <mailto:[email protected]>?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
