Skip to content

net10 replacing net9 for now - #35

Open
NickJosevski wants to merge 18 commits into
octopus/masterfrom
nj/net10-replacing-9-only
Open

NickJosevski wants to merge 18 commits into
octopus/masterfrom
nj/net10-replacing-9-only

Conversation

@NickJosevski

Copy link
Copy Markdown

No description provided.

zentron and others added 18 commits July 18, 2025 22:02
…uilding unknown exception types

The exception constructors in GitErrorsToLibGit2SharpExceptions make use of the error category by passing it through to the NativeException(string message, GitErrorCategory category) constructor which adds ("libgit2.category", category) to Data on System.Exception.

We were calling new LibGit2SharpException(m, c), but this was resolving to the string formatting constructor on LibGit2SharpException, because it does not have a constructor that takes a category. This runs a string format, so if the errorMessage contained any curly braces, that constructor would throw an System.FormatException.

This has been changed to just use the message constructor (with no format arguments), and will drop the error code (which is always Unknown anyway).
Re-introduce .net6 while we still have Calamari running in .net6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants