search

EvotecIT / OfficeIMO

Fast and easy to use cross-platform .NET library that creates or modifies Microsoft Word (DocX) and later also Excel (XLSX) files without installing any software. Library is based on Open XML SDK
MIT License
280 stars 50 forks source link

Provide multi platform project config #79

Closed byteSamurai closed 1 year ago

byteSamurai commented 1 year ago

This PR closes #77

I could to the testing on MacOS for VSC and VS:

image image

My environment:

❯ dotnet sdk check
.NET SDKs:
Version      Status
----------------------------------------
5.0.408      .NET 5.0 is out of support.
6.0.403      Up to date.
7.0.100      Up to date.

.NET Runtimes:
Name                          Version      Status
----------------------------------------------------------------------
Microsoft.AspNetCore.App      5.0.17       .NET 5.0 is out of support.
Microsoft.NETCore.App         5.0.17       .NET 5.0 is out of support.
Microsoft.AspNetCore.App      6.0.11       Up to date.
Microsoft.NETCore.App         6.0.11       Up to date.
Microsoft.AspNetCore.App      7.0.0        Up to date.
Microsoft.NETCore.App         7.0.0        Up to date.

@PrzemyslawKlys could you check for Windows, please? Also, I consider it questionable to provide support for EOL environments, especially since we are generating Office documents with this project. WDYT?

byteSamurai commented 1 year ago

Also, in my opinion, the support for Linux and macOS seems to be quite the same. So I would consider it as covered with testing macOS.

byteSamurai commented 1 year ago

(Oh and a release sooner or later would be nice 😅 )

PrzemyslawKlys commented 1 year ago

That shows up when I open your PR

image

Had to reopen VS for it to go away. Then when reopened I've run tests, and it seems to skip net 5.0 and net 6.0 and lost net 7.0 completely.

image

When I switched from Release to Debug it tested all of them

image

Did you remove 7? :)

image

byteSamurai commented 1 year ago

Did you remove 7? :)

Well, well, if you want to submit a contribution on the last minutes before the gym 🙄 Copy-Pasta-Error. Could you check again please?

PrzemyslawKlys commented 1 year ago

I've added missing 7.0 in few places, and removed 5.0.

byteSamurai commented 1 year ago

@PrzemyslawKlys sry, maybe my day was too long yesterday 😓

PrzemyslawKlys commented 1 year ago

Don't worry about it. I appreciate any effort on a project. It saves my time doing this myself, so I can't complain.