diff mbox series

README: Add note about testing with Travis CI

Message ID 20190605135641.1848-1-pvorel@suse.cz
State Changes Requested
Headers show
Series README: Add note about testing with Travis CI | expand

Commit Message

Petr Vorel June 5, 2019, 1:56 p.m. UTC 
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 README.md | 3 +++
 1 file changed, 3 insertions(+)

Comments

Cyril Hrubis June 5, 2019, 3:35 p.m. UTC | #1 
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
Petr Vorel June 5, 2019, 3:59 p.m. UTC | #2 
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
Cyril Hrubis June 6, 2019, 8:32 a.m. UTC | #3 
Hi!
> Thanks, good idea. Acking this?

Looks good, acked.
Petr Vorel June 6, 2019, 8:47 a.m. UTC | #4 
Hi Cyril,

> > Thanks, good idea. Acking this?

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

Kind regards,
Petr
diff mbox series

Patch

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).