Skip to content
This repository was archived by the owner on Dec 27, 2023. It is now read-only.

Migrated project to .NET Standard 2.0 #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

SuNNjek
Copy link

@SuNNjek SuNNjek commented Oct 23, 2019

Hi,

since this project is still using .NET 4.0 I've taken the liberty to migrate it to .NET Standard 2.0, making it run cross-platform.

I've also made some changes to the test project to make it run in the latest Visual Studio and modified the .gitignore to make it ignore all the VS related files.

@castarco
Copy link
Contributor

castarco commented Oct 23, 2019

Hi @SuNNjek , I'm wondering if you are interesting on taking the ownership of the project. I'm not maintaining it, I don't have much time, and actually I'm not using the library anymore (it was used in a proprietary project some time ago, but I moved to another job).

Relative to the PR, I guess you checked that the library can be properly compiled and linked, right?

P.D.: I even lost the signing key that I used to release versions of it...

@SuNNjek
Copy link
Author

SuNNjek commented Oct 24, 2019

I don't know if I'm the right person to take over this project because I admittedly don't understand the maths involved 😰 Otherwise I'd be happy to

And yes, I checked that the library compiles (and since CI works again, Travis agrees with me there :P). Creating a NuGet package also still works.

@DillyGoatGruff
Copy link

Hi @SuNNjek, I actually work a lot in color spaces and understand the math. However I am very new to Github so I would not feel comfortable taking ownership (yet at least). I know your changes are from a few years ago but if you were still interested in this project I would be willing to work on it with you and if you are not wanting the responsibility, I could take ownership after a period of time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants