Skip to content

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

Open
Mic92 wants to merge 1 commit intolab132:masterfrom
Mic92:forks
Open

reporter: create status in the base repository of a pull requests#42
Mic92 wants to merge 1 commit intolab132:masterfrom
Mic92:forks

Conversation

@Mic92
Copy link
Copy Markdown

@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
Copy Markdown
Author

Mic92 commented May 27, 2024

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

@Mic92
Copy link
Copy Markdown
Author

Mic92 commented May 31, 2024

Friendly ping

@pampersrocker
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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