Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Readd .travis.yml #1297

Merged
merged 2 commits into from
Jan 3, 2015
Merged

Conversation

monkeydave
Copy link
Contributor

This file was removed in commit 95429a4

At the moment the Travis-Ci build passing image points to a build from 5 days ago as there is no longer a .travis.yml file in the repo.

This file isn't the same as the one removed the other day, I have changed the travis.yml file to use ino the command line compiler for Arduino as I couldn't easily get the Makefile to work with the U8glib library so I decided to use ino instead.

It should now be easy to add commands to the script: part of the travis.yml file and check compilation with different boards and with extra options defined in Configuration.h and Configuration_adv.h whch will hopefully create a more stable Marlin going forward. The previous travis.yml file only checked the default config with an ultimaker board, its never going to pick up most of the compile errors when people are adding in stuff that doesn't get compiled in automatically with the default configuration.

Check https://travis-ci.org/monkeydave/Marlin/builds for the build history of my branch as it will fail at the moment because I had to change the directory to ErikZalm so it will compile when/if it is merged.

I am happy to add more compile options but didn't want to waste my time if this doesn't get merged in.

@boelle
Copy link
Contributor

boelle commented Jan 3, 2015

So for me the house elf that means that travis can now check more than one config... in the end avoiding to long testing rounds?

@boelle
Copy link
Contributor

boelle commented Jan 3, 2015

if i get it right i'm to blame again for its removal.... we do have a list of testers and what hardware they use so could have it check against those ? their test will just be an extra check and we can reduce test time from dev to stable to say 1 month or less maybe

@monkeydave
Copy link
Contributor Author

New features will still need to be tested to check they still work as the developer intended. This only checks that Marlin builds successfully but we can add as many different configurations as we like.

@boelle
Copy link
Contributor

boelle commented Jan 3, 2015

of course new stuff will have to be tested.... but non the less this will take some load of testers and we have an idea if a new feature will break more than it adds

so merging this one...

boelle added a commit that referenced this pull request Jan 3, 2015
@boelle boelle merged commit b5c6f11 into MarlinFirmware:Development Jan 3, 2015
@monkeydave
Copy link
Contributor Author

Ok I will add more configurations to the travis.yml file and submit another pull request, probably tomorrow before I get around to it.

@boelle
Copy link
Contributor

boelle commented Jan 3, 2015

no rush, we all work on this for fun 😄

Den lørdag den 3. januar 2015 skrev monkeydave notifications@github.com:

Ok I will add more configurations to the travis.yml file and submit
another pull request, probably tomorrow before I get around to it.


Reply to this email directly or view it on GitHub
https://github.com/ErikZalm/Marlin/pull/1297#issuecomment-68595075.

@alexborro
Copy link
Contributor

Thanks @monkeydave , automatic compilation tests are ALWAYS welcome.

If you have more free time, please, add more tests. It will help us all making a more stable Marlin.

Cheers.

Alex.

@monkeydave monkeydave deleted the Readd-Travis-CI branch January 3, 2015 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants