Přemek Vysoký
Přemek Vysoký
I think this might be handled with #4407 as that will touch this area too
I think it's alright to keep it as-is. I think this issue might as easy as just using the `CodeFlow` type to convey the previous flow, name it appropriately and...
1. We now have 3 attempts. 2. We tried this in https://github.com/dotnet/arcade-services/pull/4421 but it would not actually help us that much, if we paused everything at once and then slammed...
Closing in favour of https://github.com/dotnet/arcade-services/issues/4635
@mmitche see my comment in Teams about ignored checks and then merge this if you still feel like we should do this
I guess we need to check the GitHub API response to see what is the actual value it returns
The GraphQL gives me this: ``` "checkRuns": { "nodes": [ { "name": "sdk-unified-build", "status": "COMPLETED", "conclusion": "NEUTRAL" }, } ```
@mmitche are we worried that `Neutral` is also the state before we queue the check? I remember some worries like that so that the service does not merge stuff right...
https://github.com/dotnet/arcade-services/pull/4026
@dkurepa somehow the DI test is hanging in CI but goes green locally. I didn't have time to fix this but I finished your PR and merged it without it.