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

Added Travis-CI and Cirrus-CI builds #83

Merged
merged 25 commits into from
Mar 27, 2019
Merged

Conversation

mmcc007
Copy link
Contributor

@mmcc007 mmcc007 commented Mar 25, 2019

Since examples are dependent on version of bash, I installed bash 4.4.

It's pretty easy to configure Travis to run examples against a matrix of bash versions if necessary in future.

@mmcc007
Copy link
Contributor Author

mmcc007 commented Mar 25, 2019

BTW: You can see build on my fork here:
https://travis-ci.com/mmcc007/bash-oo-framework/builds/105608466

@mmcc007
Copy link
Contributor Author

mmcc007 commented Mar 25, 2019

I added builds on Cirrus-CI which is running Bash 4.4.12 for much better results:
https://cirrus-ci.com/task/5121293144817664
Can also add a matrix of bash versions for testing on Cirrus-CI.

@mmcc007 mmcc007 changed the title Added Travis-CI builds Added Travis-CI and Cirrus-CI builds Mar 25, 2019
@mmcc007
Copy link
Contributor Author

mmcc007 commented Mar 26, 2019

Added matrix and official docker builds of bash for both Cirrus and Travis. To seem them running:
https://cirrus-ci.com/github/mmcc007/bash-oo-framework
https://travis-ci.com/mmcc007/bash-oo-framework
Thanks to @fkorotkov for assistance on Cirrus-CI

@niieani niieani merged commit 3abbb6e into niieani:master Mar 27, 2019
@niieani
Copy link
Owner

niieani commented Mar 27, 2019

Thank you so much @mmcc007 and @fkorotkov!

@mmcc007
Copy link
Contributor Author

mmcc007 commented Mar 27, 2019

Sure, no problem. Might be using your logging module in another project.

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