Thomas Desveaux

Sorry for the late reply. I don't think I'll have the time to setup a test env for this before some time. Will this be in the next Sentry release?...

Had some time and tested. Without the updated dependency: `-052][569]LogNiagara: Compiling System NiagaraSystem` With the update: `Compiling System NiagaraSystem` So not sure my `ERROR_NO_MORE_ITEMS / 0x0` is really related but...

@p12tic apologize about the ping, but I feel I need to confirm an architectural / code style choice with you. I started to look into this by refactoring the code...

This would help a lot with type hints as `inlineCallback` function return types are not really straightforward. They need to be typed with `Generator[Deferred[Any], Any, _ReturnType]` (maybe Twisted provide a...

EDIT: Forgot to set `c['collapseRequests'] = False` in my repro case, so the BuildRequest was simply collapsed. There still is an issue were the build request that is skipped is...

Tried to implement a fix for this here: https://github.com/tdesveaux/buildbot/tree/source/git/use-change-from-targetted-repository Seeing the amount of changes required, maybe rather than considering this a bug, implementing a new `GitClone` step would be more...

Been a while, and I don't remember exactly all. So I'll showcase the issue showed in the first post. On this build: https://builder.blender.org/admin/#/builders/136/builds/2402 You can see `trigger 3 builds, 1...

@mrexox Hi, I looked a bit and I believe the issue is that `git lfs pre-push` expect to read commit information on STDIN. This is documented, here is the first...

Tried adding a dumb `cmd.Stdin = os.Stdin` in execute_unix.go `RawExecute` and it works now. I believe it's not a proper solution as if another hook needs the information from stdin,...

If I can suggest a quick "fix" for the moment. Since you already have the [`use_stdin`](https://github.com/evilmartians/lefthook/blob/master/docs/configuration.md#use_stdin) option for commands, you could throw an error if a pre-push command has it...