mperham/sidekiq — pull request #1382 by petergoldstein (2013-12-05 17:24:59+00:00)
Title: Remove branches and notifications directives from .travis.yml

I'm not sure if these are here deliberately, or only as a vestige from back in the day, but I'd like to suggest that these directives be removed.

One of the big plusses for Travis is the ability to do CI runs on forked repos and PRs. With the branches restriction this doesn't happen unless development occurs on the master branch. In practice this means that I always wind up making a commit that removes these directives and then rolling it back when I submit a PR.

At the same time we don't want to notify the IRC channel or mailing list every time there's a failed build on a forked repo.

So I suggest removing both these directives. @mperham - your thoughts?