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

reporter: create status in the base repository of a pull requests #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Mic92
Copy link

@Mic92 Mic92 commented May 27, 2024

Buildbot might not have access to the build status of the fork and for pull request we want the information in the pull request window rather than the commit on the fork.

Buildbot might not have access to the build status of the fork
and for pull request we want the information in the pull request
window rather than the commit on the fork.
@Mic92
Copy link
Author

Mic92 commented May 27, 2024

Here is an example: https://git.clan.lol/clan/clan-core/pulls/1458

@Mic92
Copy link
Author

Mic92 commented May 31, 2024

Friendly ping

@pampersrocker
Copy link
Contributor

Should this case then handled properly? This PR just removes this info in all cases as far as I can tell.

@Mic92
Copy link
Author

Mic92 commented Sep 28, 2024

This is handled now properly from my perspective. Before it was trying to access information, which is not available if the pull request comes from a fork.

@Mic92
Copy link
Author

Mic92 commented Jan 5, 2025

This has been tested by multiple parties for quite a while now.

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.

2 participants