Jan Meiswinkel

Results 31 comments of Jan Meiswinkel

Good point, thanks. About the other point: Do you mean unsandboxed pipelines that run potentially dangerous functions that have been put on the allowlist?

> > Do you mean unsandboxed pipelines that run potentially dangerous functions that have been put on the allowlist? > > No, users can choose to disable the sandbox for...

> Our general policy is to review proposed API additions along with the proposed consumers. This can help us better understand the intended use case in order to properly review...

Hey @basil, Please let me know if this PR needs further justification. Thanks!

Thanks for your review, @res0nance. Very good point! My idea was to get the user in the listener implementation. It looked like the User object was still accessible there but...

Hi, I will be very busy for the next two weeks and probably not able to work on this PR during that time. Please feel free to continue reviewing but...

Hey @res0nance, sorry for the delay. I implemented the requested change. Thanks for the hint! PR is ready for review again.

@mawinter69 @basil Do you want to review this PR or should I tag the reviewer team?

Is this PR ready for merge? :) If anything else should be done on my side, please let me know.

> Please add a changelog entry Done