[Qemu-devel] [PATCH 04/18] Acceptance tests: fix doc reference to avocado_qemu directory

Cleber Rosa posted 18 patches 7 years ago
Maintainers: "Alex Bennée" <alex.bennee@linaro.org>, Eduardo Habkost <ehabkost@redhat.com>, Cleber Rosa <crosa@redhat.com>, Fam Zheng <fam@euphon.net>, "Philippe Mathieu-Daudé" <philmd@redhat.com>
There is a newer version of this series
[Qemu-devel] [PATCH 04/18] Acceptance tests: fix doc reference to avocado_qemu directory
Posted by Cleber Rosa 7 years ago
The "this directory" reference is misleading and confusing, it's a
leftover from when this text was proposed in a README file inside
the "tests/acceptance/avocado_qemu" directory.

When that text was moved to the top level docs directory, the
reference was not updated.

Signed-off-by: Cleber Rosa <crosa@redhat.com>
---
 docs/devel/testing.rst | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
index 18e2c0868a..44c9b3ae74 100644
--- a/docs/devel/testing.rst
+++ b/docs/devel/testing.rst
@@ -590,8 +590,9 @@ Alternatively, follow the instructions on this link:
 Overview
 --------
 
-This directory provides the ``avocado_qemu`` Python module, containing
-the ``avocado_qemu.Test`` class.  Here's a simple usage example:
+The ``tests/acceptance/avocado_qemu`` directory provides the
+``avocado_qemu`` Python module, containing the ``avocado_qemu.Test``
+class.  Here's a simple usage example:
 
 .. code::
 
-- 
2.20.1


Re: [Qemu-devel] [PATCH 04/18] Acceptance tests: fix doc reference to avocado_qemu directory
Posted by Caio Carrara 7 years ago
On Thu, Jan 17, 2019 at 01:56:14PM -0500, Cleber Rosa wrote:
> The "this directory" reference is misleading and confusing, it's a
> leftover from when this text was proposed in a README file inside
> the "tests/acceptance/avocado_qemu" directory.
> 
> When that text was moved to the top level docs directory, the
> reference was not updated.
> 
> Signed-off-by: Cleber Rosa <crosa@redhat.com>

Reviewed-by: Caio Carrara <ccarrara@redhat.com>

> ---
>  docs/devel/testing.rst | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> index 18e2c0868a..44c9b3ae74 100644
> --- a/docs/devel/testing.rst
> +++ b/docs/devel/testing.rst
> @@ -590,8 +590,9 @@ Alternatively, follow the instructions on this link:
>  Overview
>  --------
>  
> -This directory provides the ``avocado_qemu`` Python module, containing
> -the ``avocado_qemu.Test`` class.  Here's a simple usage example:
> +The ``tests/acceptance/avocado_qemu`` directory provides the
> +``avocado_qemu`` Python module, containing the ``avocado_qemu.Test``
> +class.  Here's a simple usage example:
>  
>  .. code::
>  
> -- 
> 2.20.1
> 

-- 
Caio Carrara
Software Engineer, Virt Team - Red Hat
ccarrara@redhat.com

Re: [Qemu-devel] [PATCH 04/18] Acceptance tests: fix doc reference to avocado_qemu directory
Posted by Philippe Mathieu-Daudé 7 years ago
On 1/17/19 7:56 PM, Cleber Rosa wrote:
> The "this directory" reference is misleading and confusing, it's a
> leftover from when this text was proposed in a README file inside
> the "tests/acceptance/avocado_qemu" directory.
> 
> When that text was moved to the top level docs directory, the
> reference was not updated.
> 
> Signed-off-by: Cleber Rosa <crosa@redhat.com>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> ---
>  docs/devel/testing.rst | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> index 18e2c0868a..44c9b3ae74 100644
> --- a/docs/devel/testing.rst
> +++ b/docs/devel/testing.rst
> @@ -590,8 +590,9 @@ Alternatively, follow the instructions on this link:
>  Overview
>  --------
>  
> -This directory provides the ``avocado_qemu`` Python module, containing
> -the ``avocado_qemu.Test`` class.  Here's a simple usage example:
> +The ``tests/acceptance/avocado_qemu`` directory provides the
> +``avocado_qemu`` Python module, containing the ``avocado_qemu.Test``
> +class.  Here's a simple usage example:
>  
>  .. code::
>  
>