Tim MacDonald

Results 87 comments of Tim MacDonald

@TimWolla I hope you don't mind, I decided to add some more improvements to this PR. I think these are the last of these kinda of markup improvements I wanna...

On hold until https://github.com/laravel/framework/pull/45492 is merged.

This won't work on all version of PHP that Inertia currently supports. Inertia currently supports 7.2+. The spread operator was introduced in PHP 7.4

Windows build is failing cause of permission issues. I’ll figure that out…maybe.

@woodspire I'm not sure I totally follow, sorry. Could you provide some code examples of what you are thinking?

@devajmeireles I built that and ripped it out to make Context as simple as possible. I'll consider bringing that back, but can you think of a specific real-world example of...

@aran112000, I'm not intimately familiar with SQS, but if the trace ID can be set already in Laravel then the answer is: yes! If not, could be a good PR...

@tpetry, you can perform work and reset all context with `Context::flush()`. You may also forget specific keys by calling `Context::forget([/* ... */)` and `Context::forgetHidden([/* ... */)`. If you wanted to...

`Context::scope` - dig it. Will do this instead of `Context::with`. Then you can add to context within the scope as needed. Serves both purposes well. It would inherit the current...

Gonna sit on this for a bit to make sure I'm happy with everything. Also considering adding a way to indicate that a job expects an authenticated user. This will...