-
-
Notifications
You must be signed in to change notification settings - Fork 529
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
README: fix Jenkins and travis-ci badges. #444
Conversation
[](http://ci.px4.io:8080/blue/organizations/jenkins/DroneCore/activity) | ||
[](https://travis-ci.org/dronecore/DroneCore) | ||
[](http://ci.px4.io:8080/blue/organizations/jenkins/DronecodeSDK/activity) | ||
[](https://travis-ci.org/Dronecode/DronecodeSDK) | ||
[](https://ci.appveyor.com/project/julianoes/dronecore/branch/develop) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Feel free to merge. Only question, why an @julianoes appveyer branch? https://ci.appveyor.com/project/julianoes/dronecore/branch/develop
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's just what it did when I set it up. Presumably I should have logged into appveyor as DroneCore or now Dronecode when I did it instead of julianoes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go for it. Thanks :-)
@dagar there is a problem with Jenkins notifying via webhook. Do you know how the permissions should work?
http://ci.px4.io:8080/administrativeMonitor/GitHubHookRegisterProblemMonitor/ |
Appveyor and coveralls URLs stay the same for now.
8f08fb5
to
2bb41e3
Compare
README: fix Jenkins and travis-ci badges.
Appveyor and coveralls URLs stay the same for now.
The appveyor URL just stays as far as I can see unless you tell them to change it manually.
For coveralls,
the rename doesn't work, see lemurheavy/coveralls-public#1156the rename was done manually.