02/02/2019 11:28, Hari Kumar Vemula: > --- /dev/null > +++ b/doc/guides/prog_guide/meson_ut.rst > @@ -0,0 +1,143 @@ > +.. SPDX-License-Identifier: BSD-3-Clause > + > + Copyright(c) 2018-2019 Intel Corporation.
Why inserting a blank line between SPDX and Copyright. > + > +.. _Meson: > + > +Meson_UT > +======== This is a title. Why having an underscore instead of space? Why not saying Unit Tests? Or better, "Run Unit Tests with Meson". > + > +This document describes the below list in detail. > + > +* Building and Running the unit tests. > +* Grouping of testcases. > +* How to run different test suites. > +* Support for skipped tests. This list is useless because automatically generated by Sphinx. Sorry, I stop reading here. Marko, John, did you try to review this patch?
