Zoltán Lehóczky

Results 1162 comments of Zoltán Lehóczky

https://github.com/OrchardCMS/OrchardCore/pull/17346 doesn't actually completely fix this. We still need the docs, and more importantly, as Sebastien pointed out during today's meeting, in the codegen templates: - https://github.com/OrchardCMS/OrchardCore/blob/main/src/Templates/OrchardCore.ProjectTemplates/content/OrchardCore.Templates.Cms.Web/wwwroot/.placeholder - https://github.com/OrchardCMS/OrchardCore/blob/main/src/Templates/OrchardCore.ProjectTemplates/content/OrchardCore.Templates.Mvc.Web/wwwroot/.placeholder

YesSql added Meziantou.Analyzer, you mean? That's a good analyzer package, but we shouldn't just use it from YesSql transitively. That reference needs to be changed with `analyzers` so we don't...

The build fails due to an analyzer violation to use the cancellation token feature introduced under https://github.com/sebastienros/yessql/pull/627. The violations are valid, but much like `async`, this propagates through the whole...

How about having an `ICancellationTokenAccess`, in a similar vein to `IUpdateModelAccessor`? Then, we could use `HttpContext.RequestAborted` for that by default, except in background tasks where we have another token.

Hmm, interesting that we have this, I thought it would just update the dependencies. I guess this can be useful. Since my understanding is that with the current config this...

So far so good. For GitHub Actions actions it can also update releases by pinning them to commit SHAs, which is the recommendation since tags can be moved: https://github.com/OrchardCMS/OrchardCore/pull/17147.

I've triggered all updates. Reviewed and tested those that I could, also merged them with manual fixed if necessary. A couple of them are up-to-date where I need input/review from...

No and no. It's not an ideal solution, but I guess the only thing the Revolut developers could come up with to allow everyone with read access to the repo...

I guess only admins can use that, then, so that's why we have this issue.

`_session.CancelAsync()` will work anywhere, also in a handler. But I don't think it's too nice to cancel the transaction if we can better. However, what's the scenario that this would...