search

madelson / MedallionShell

A .NET library simplifying the execution and chaining of processes
MIT License
415 stars 32 forks source link

Clean up Mono support and replace it with robust .NET 8 support #119

Open Bartleby2718 opened 6 months ago

Bartleby2718 commented 6 months ago

Filed per https://github.com/madelson/MedallionShell/pull/111#issuecomment-2133513165

Bartleby2718 commented 6 months ago

@madelson

  1. Given https://github.com/madelson/MedallionShell/blob/release-1.7/MedallionShell/WindowsCommandLineSyntax.cs#L15-L16, it seems like we can drop https://github.com/madelson/MedallionShell/blob/release-1.7/MedallionShell/MonoUnixCommandLineSyntax.cs entirely. Does that sound right?
  2. https://github.com/madelson/MedallionShell/blob/release-1.7/MedallionShell/PlatformCompatibilityHelper.cs#L51-L76 mentions Mono-specific issues. Can this now be ignored?