sphinx-doc / alabaster

Lightweight, configurable Sphinx theme
http://alabaster.readthedocs.io/
Other
735 stars 189 forks source link

[Bug] Option to specify the Travis platform #145

Open MrinalJain17 opened 6 years ago

MrinalJain17 commented 6 years ago

Traditionally, Travis-ci provides 2 platforms: travis-ci.com - for private/paid repos and travis-ci.org - for free repos. But according to this announcement, all projects from .org will move to .com eventually.

The problem is that when setting travis_button=True, the badge displays the build status of the repo from travis-ci.org, which might not be the case for all the users. For someone having the repo activated on travis-ci.com, the badge will not show the proper build status.

For instance, in my case, the build was passing, but the badge showed unknown - since my repo is activated on travis-ci.com and not on travis-ci.org.

JurajMa commented 6 years ago

Same problem here.

webknjaz commented 5 years ago

Ah.. was just about to create the same issue.

Occurrences in two docs sites I maintain (there's much more):

And this misleading button was there since May.. This needs to be fixed ASAP. I'm going to submit a PR.

@bitprophet ^

webknjaz commented 5 years ago

Done: https://github.com/bitprophet/alabaster/pull/147

UPD: If somebody is looking to monkey-patch
it in their project while this isn't
merged — just apply this 👇

                https://github.com/sanitizers/octomachinery/commit/e5756d9ad21cb77ed138bcf865eea0eb9efcd03b