-
Notifications
You must be signed in to change notification settings - Fork 219
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
Show AppVeyor Status in pull requests #301
Comments
AppVeyor is active for pull_request and push, see https://ci.appveyor.com/project/icsharpcode/codeconverter/history |
Thanks. I've updated the issue - didn't realise we already had CI, it wasn't visible! You can set it up to show on PRs, eg. libpnet/libpnet#363 (comment) (ctrl-f for "check", or scroll to the bottom). I can't remember how I set this up for libpnet, I assume I clicked through some UI on AppVeyor's side. |
It still works for refactoring essentials e.g., but has never worked on this repo since it was split out. Probably just needs a webhook approximately here to have the right settings the same as refactoring essentials: |
I have asked @dgrunwald what the magic is |
I don't remember any magic. Maybe some permission problem? Setting statuses requires some form of write access. I don't remember how I set that up with AppVeyor, I guess there was some OAuth stuff involved... |
Thanks, that was exactly the hint I needed - the build server GH account had only read access, whereas in all other repos it had write. |
It would be great if the AppVeyor status showed in pull requests.
Status can be seen here, but not on the PR itself - https://ci.appveyor.com/project/icsharpcode/codeconverter/history
The text was updated successfully, but these errors were encountered: