OSS GH repo, appears to link to the wrong appveyor URL

Stefan Penner's Avatar

Stefan Penner

21 May, 2015 03:57 PM

for some reason, all ember-cli appveyor links on GH are pointing to https://ci.appveyor.com/project/jayphelps/ember-cli/build/ notice jayphelps and not ember-cli.

example https://github.com/ember-cli/ember-cli/pull/4104

  1. Support Staff 1 Posted by Feodor Fitsner on 21 May, 2015 04:26 PM

    Feodor Fitsner's Avatar

    Well, there is ember-cli project exists under jayphelps account. Apparently, if it's changing PR status then it has write permissions to do that. Do you know who are they?

  2. 2 Posted by Stefan Penner on 21 May, 2015 05:04 PM

    Stefan Penner's Avatar

    Sorry, can you elaborate on" Apparently, if it's changing PR status" ?

  3. 3 Posted by Stefan Penner on 21 May, 2015 05:06 PM

    Stefan Penner's Avatar

    interestingly, tests do continue to run correctly on https://ci.appveyor.com/project/embercli/ember-cli/history just the jayphelps one is reporting back to GH. Seems funky...

  4. Support Staff 4 Posted by Feodor Fitsner on 21 May, 2015 05:08 PM

    Feodor Fitsner's Avatar

    That example pull request - AppVeyor status navigates to https://ci.appveyor.com/project/jayphelps/ember-cli/build/406.

    There is another ember-cli project exists in AppVeyor under jayphelps account. If that project is able to update commit status then GitHub username authorized in AppVeyor for that account has permissions to do that. Most probably they override your original status coming from builds under embercli account.

  5. Support Staff 5 Posted by Feodor Fitsner on 21 May, 2015 05:09 PM

    Feodor Fitsner's Avatar

    If you don't see statuses set by builds from embercli account then GitHub username authorized for embercli account doesn't have permissions to put that status on that repo.

  6. 6 Posted by Stefan Penner on 21 May, 2015 05:10 PM

    Stefan Penner's Avatar

    Yes, jayphelps has commit to embercli/ember-cli on gh.

    Still this seems strange, as it never happens with travis?

    Anyways, how do we fix this and prevent it from happening in the future.

  7. Support Staff 7 Posted by Feodor Fitsner on 21 May, 2015 05:13 PM

    Feodor Fitsner's Avatar

    Leave only one project in AppVeyor building ember-cli?

  8. 8 Posted by Stefan Penner on 21 May, 2015 06:12 PM

    Stefan Penner's Avatar

    That may be a short term solution. But forks should be able to run there own CI, no?

  9. Support Staff 9 Posted by Feodor Fitsner on 21 May, 2015 07:30 PM

    Feodor Fitsner's Avatar

    Yes, that's correct - people should be able to build forks without interfering with base repo. But that project under jayphelps account builds main ember-cli/ember-cli repo, not a fork.

  10. 10 Posted by Stefan Penner on 21 May, 2015 08:05 PM

    Stefan Penner's Avatar

    ah, that makes sense. Thank you

  11. 11 Posted by Jay Phelps on 22 May, 2015 01:10 AM

    Jay Phelps's Avatar

    resolved.

  12. Ilya Finkelshteyn closed this discussion on 25 Aug, 2018 01:56 AM.

Comments are currently closed for this discussion. You can start a new one.

Keyboard shortcuts

Generic

? Show this help
ESC Blurs the current field

Comment Form

r Focus the comment reply box
^ + ↩ Submit the comment

You can use Command ⌘ instead of Control ^ on Mac