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

Extend CI tests with older c++ standards #273

Merged
merged 7 commits into from
Feb 9, 2016

Conversation

niosHD
Copy link
Contributor

@niosHD niosHD commented Feb 7, 2016

Hi!
This PR increases the test coverage by building and testing the library in c++ 98, 03, and 0x mode with Travis. The older standards are only tested in release mode to reduce the number of builds.

This informs travis that the container-based build environment can
not be used.
Given that it is required for header only builds it has to be
installed too.
@vitaut
Copy link
Contributor

vitaut commented Feb 8, 2016

Thanks for another great PR. Mostly looks good, just a small comment above.

@niosHD
Copy link
Contributor Author

niosHD commented Feb 9, 2016

Sure, thank you!

vitaut added a commit that referenced this pull request Feb 9, 2016
Extend CI tests with older c++ standards
@vitaut vitaut merged commit c7b7141 into fmtlib:master Feb 9, 2016
@vitaut
Copy link
Contributor

vitaut commented Feb 9, 2016

Merged with small changes in b8c6192. Thanks again!

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.

2 participants