Tim MacDonald
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...