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

Add travis config file #315

Merged
merged 1 commit into from
Mar 19, 2016
Merged

Add travis config file #315

merged 1 commit into from
Mar 19, 2016

Conversation

xantares
Copy link
Contributor

@xantares xantares commented Sep 1, 2015

travis continuous integration service must be enabled from repo settings

@smortex
Copy link
Contributor

smortex commented Sep 1, 2015

Doesn't it make sense to install cutter as a dependency and run the test suite shipped with libnfc?

I understand that checking that the software builds is a good start (-Wall -Wextra -Werror -pedantic may be advisable in such a case however), but building successfully a broken software does not really help I think.

@xantares
Copy link
Contributor Author

xantares commented Sep 1, 2015

I know it's not getting better than that

@neomilium
Copy link
Member

Any improvement ?

neomilium added a commit that referenced this pull request Mar 19, 2016
@neomilium neomilium merged commit bc8bb8d into nfc-tools:master Mar 19, 2016
@neomilium
Copy link
Member

Thanks. Hope someone will improve it later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants