vincentlaucsb / csv-parser

A high-performance, fully-featured CSV parser and serializer for modern C++.
MIT License
901 stars 150 forks source link

Add Clang builds #178

Closed xgdgsc closed 3 years ago

xgdgsc commented 3 years ago

Please see https://app.travis-ci.com/github/xgdgsc/csv-parser/builds/233738949 for various build errors on gcc/clang. Why isn' t travis working at your master branch?

vincentlaucsb commented 3 years ago

I have no idea. When travis-ci moved from .org to .com I had to recreate my account and for some reason all of my repos with a .travis.yml aren't recognized.

The build errors are due to another PR that brought in some very restrictive error flags when IF (UNIX) is true. I might remove a few, while fixing a few others. If you remove the flags however, the library should build.

xgdgsc commented 3 years ago

You might need to enable repo at https://app.travis-ci.com/dashboard

xgdgsc commented 3 years ago

Or at settings https://app.travis-ci.com/account/repositories , click " Manage repositories on GitHub"

vincentlaucsb commented 3 years ago

That's the problem. It shows all of my repos except for the ones that I actually used with Travis in the past.

xgdgsc commented 3 years ago

https://travis-ci.com/github/vincentlaucsb/csv-parser what if you access this url?

vincentlaucsb commented 3 years ago

image 🤷

vincentlaucsb commented 3 years ago

This is a good PR and something I've wanted to do for a while. I created a ticket with the Travis CI team and hopefully they can fix this.

vincentlaucsb commented 3 years ago

I've just migrated my repos over from travis-ci.org to travis-ci.com. Let's see what happens.