.NET

I'm a bit confused about how DistributedSession CommitAsync should work. The documentation states that it may throw if the data store is unavailable.

However, I see it

The "src" directory contains a number of JS files that are added to the base copy of RN. It is not a complete source tree itself. Renaming to "patches" would increase clarity of intent, and follow OSS "patch-package" conventions.

using \d n regexes does not perform as expected, and in many cases [0-9] should be preferred (as discovered here).

• Review existing uses of \d and replace with [0-9] where appropriate
• Consider adding this to the docs somewhere. Not sure where this would be appropriate to document, perhaps some discussion

What's the issue you encountered?

Fire Emblem: Three Houses is a single-player game, but it has several asynchronous multiplayer features. Among them is the possibility, during weekly planning scenes (the 'calendar' screen), to look up what other players have done on the current week. This is entirely reliant on Nintendo's servers and completely useless to anyone playing on Ryujinx, but eve

We can build much smaller Mono runtime for .NET Core purposes by simply removing code we don't need in this configuration. We did a few easy initial steps but we can go much further.

This list is not comprehensive but parts like

• Culture Data
• Any PAL related code
• Unused icalls

This will help be explicit about which SDK version is expected, and should make CI/CD builds behave more consistently.

Let's make the error message more actionable.

Error user sees:

I would recommend adding similar named column(s):

- $"Provided {columnPurpose} column '{columnName}' not found in training data."
+ $"Provided {columnPurpose} column '{columnName}' not found in training 

Various characters (e.g. |) if used in a Params will end up invalidating the generated markdown, causing it to render incorrectly. It'd be helpful if benchmarkdotnet could escape the markdown output, e.g. output \| instead of |.