[LTP] [PATCH] README: Add note about testing with Travis CI

[LTP] [PATCH] README: Add note about testing with Travis CI

[Petr Vorel]

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 README.md | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/README.md b/README.md
index 1eb4747c3..c017b5032 100644
--- a/README.md
+++ b/README.md
@@ -185,3 +185,6 @@ list. Also note that these documents are available online at:
 * https://github.com/linux-test-project/ltp/wiki/C-Test-Case-Tutorial
 
 Although we accept GitHub pull requests, the preferred way is sending patches to our mailing list.
+
+It's recommended to test patches on Travis CI before posting to mailing list
+(sign up to Travis CI and enable running builds on your LTP fork on https://travis-ci.org/account/repositories).
-- 
2.21.0

[LTP] [PATCH] README: Add note about testing with Travis CI

[Cyril Hrubis]

Hi!
> +It's recommended to test patches on Travis CI before posting to mailing list

Maybe we should explain why this is a good idea, something as:

It's a good idea to test patches on Travis CI before posting to mailing
list. Our travis setup covers various architectures and distributions in
order to make sure LTP compiles cleanly on most common configurations.

> +(sign up to Travis CI and enable running builds on your LTP fork on https://travis-ci.org/account/repositories).
> -- 
> 2.21.0
> 
> 
> -- 
> Mailing list info: https://lists.linux.it/listinfo/ltp

-- 
Cyril Hrubis
chrubis@suse.cz

[LTP] [PATCH] README: Add note about testing with Travis CI

[Petr Vorel]

Hi Cyril,

> Maybe we should explain why this is a good idea, something as:

> It's a good idea to test patches on Travis CI before posting to mailing
> list. Our travis setup covers various architectures and distributions in
> order to make sure LTP compiles cleanly on most common configurations.
Thanks, good idea. Acking this?

+
+It's a good idea to test patches on Travis CI before posting to mailing
+list. Our travis setup covers various architectures and distributions in
+order to make sure LTP compiles cleanly on most common configurations.
+For testing you need to sign up to Travis CI, enable running builds on your LTP fork on
+https://travis-ci.org/account/repositories and push your branch.


Kind regards,
Petr

[LTP] [PATCH] README: Add note about testing with Travis CI

[Cyril Hrubis]

Hi!
> Thanks, good idea. Acking this?

Looks good, acked.

-- 
Cyril Hrubis
chrubis@suse.cz

[LTP] [PATCH] README: Add note about testing with Travis CI

[Petr Vorel]

Hi Cyril,

> > Thanks, good idea. Acking this?

> Looks good, acked.
Thanks! This (unsent v2 version) merged.

Kind regards,
Petr