allow successful TravisCI runs in forks

[jmhodges]

Most forks won't be in goveralls and so this command in travis.yml was,
previously, failing and causing the build to fail.

Now, it doesn't!

Checklist

• Code compiles correctly
• Created tests which fail without the change (if possible)
• All tests passing
• Extended the README / documentation, if necessary
• Added myself / the copyright holder to the AUTHORS file

[jmhodges]

Gentle ping!

[methane]

I think it's very common issue and there should be official recommendation
or de-facto standard.

Is || exit 0 idiom is recommended officially? Or widely used?
Is there no better way?

[methane]

Maybe, moving goveralls to after_script?

https://docs.travis-ci.com/user/customizing-the-build/#Breaking-the-Build

The exit code of after_success, after_failure, after_script and subsequent stages do not affect the build result. However, if one of these stages times out, the build is marked as a failure.

[jmhodges]

Okay, did that

[julienschmidt]

@jmhodges if you implemented this change already, please push your commits so that we can merge the fix.

[jmhodges]

Ah, word, mispushed earlier. It's up here now.

[jmhodges]

Thanks!