Add Continuous Integration about rust-decimal HOT 6 CLOSED

I've now set it up so that PR's into master require TravisCI builds to pass. Now that TravisCI builds do pass I think we can close this issue.

from rust-decimal.

We could also add a PR webhook here so that only PR's that pass the build check can be merged. I'm more inclined to do that and continue using tags for releases than using develop however am open for discussion here. By the way, the master branch is currently protected against force merges etc so a PR is (now) required in order to get something into master (hence why the PR webhook could work).

from rust-decimal.

I would definitely set up Github to refuse pull requests until they pass the CI tests; then it's up to you whether you want to use a develop branch. It's always possible to create new branches as need be.

from rust-decimal.

Also, have added to the Readme page.

from rust-decimal.

I'm new to using Travis so I may have set this up incorrectly for forks. Is it expected that you'd need to authorize your fork on Travis also @JP-Ellis ?

from rust-decimal.

Nope, nevermind. Travis just requires conflicts to be resolved first. All good.

from rust-decimal.