RickStrahl / SetResolution

Quickly set Windows Display Resolution via Command Line
Other
137 stars 5 forks source link

Target net6.0-windows #1

Closed Arithmomaniac closed 1 year ago

Arithmomaniac commented 1 year ago

https://github.com/RickStrahl/SetResolution/blob/5c00a228ef9416e19866168bdf8eb0ff19eff8d1/SetResolution/Westwind.SetResolution.csproj#L55

Using net6.0-windows would prevent it from being available (uselessly) IIRC.

RickStrahl commented 1 year ago

Funny that occurred to me today as well. Will update tomorrow.

RickStrahl commented 1 year ago

Turns out packages don't allow for platform specific targets - so net6.0-windows is not allowed.

image

I'll add a note to the description to minimize use outside of Windows.

Arithmomaniac commented 1 year ago

🫤

In addition to the description, you could also check RuntimeInformation.IsOSPlatform(OSPlatform.Windows) on tool launch and fail fast on MacOS/Linux

RickStrahl commented 1 year ago

LOL - if somebody installs this tool with Windows in the title description and runs in on Linux, I think I'm not too worried about a nasty system error message :smile: