-
Notifications
You must be signed in to change notification settings - Fork 337
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
Add all shells to Travis CI configuration #496
Add all shells to Travis CI configuration #496
Conversation
Doing a build for each shell makes sense to find errors easier but it took almost 17 minutes for the 8 builds to complete. Doing one Linux and one Mac OS build, with all shells each, took a little under 5 minutes. What do others think? Getting a quick response is worth a lot but not knowing where the error happens isn't great. I think I would go with the combined approach for now, enjoy the better performance and see if we like it. |
Ah, on a sidenote, did you disable Travis builds for PRs, @nerdvegas? I think it would definitely help to enable those because it runs PRs through its paces and signals if the changes made in the PR break the tests. This helps you decide if it's ok to merge and helps the developer to see that there is still some work to be done right away. |
I had to do a couple more updates to the Mac OS side of things because I noticed the build failed from time to time because of the before install scripts erroring out. Feels a lot more robust now. |
On this soon @skral, cheers. |
Marking ON HOLD as we're going to be using a GitHub Actions- based CI test suite (or possible Azure Pipelines, but I'm banking on GH). This is for the purpose of readiness for ASWF adoption. |
Closed as we're now using GitHub Actions: #750 |
This adds all shells in Mac OS and Linux to the Travis CI configuration. Running
selftest
in every shell will increase our confidence in merging new code changes.While setting up the configuration I noticed tests are breaking for
sh
on Linux andtcsh
on Mac OS. I will try to take a look at why they are failing so we don't end up with a "build:error" badge right after the merge.@nerdvegas I know you haven't been a big fan of Travis CI so far but I am sure we can figure out why the badge hasn't been all green all the time.
Let me know what you think of this PR.