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

Fix broken travis branches #546

Merged
merged 1 commit into from
Oct 13, 2015
Merged

Fix broken travis branches #546

merged 1 commit into from
Oct 13, 2015

Conversation

driskell
Copy link
Contributor

Hello,

Looks like when you request a build status for a branch it ignores the branch!
Seems a typo in the code.

Jason

Hello,

Looks like when you request a build status for a branch it ignores the branch!
Seems a typo in the code.

Jason
@espadrine espadrine merged commit cc070f5 into badges:master Oct 13, 2015
@espadrine
Copy link
Member

Good lord, you're right! Thank you!

@jakirkham
Copy link

I think the original issue I was raising here ( #528 ) is addressed by this change. However, I think discussion in that issue turned to a potentially different issue so I will leave it open for now.

@jakirkham
Copy link

Is it possible the HHVM branch badge is suffering from a similar problem?

@jakirkham
Copy link

Also, TeamCity CodeBetter Coverage badge is complaining it is malformed. Wondering if a similar fix would help it.

@espadrine
Copy link
Member

Usually, those caused by a change in the format of the data. Adding a breakpoint in server.js reveals what needs fixing. But it is a game of whack-a-mole, so patches are very welcome!

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

Successfully merging this pull request may close these issues.

3 participants