Skip to content
This repository has been archived by the owner on Dec 5, 2021. It is now read-only.

Upgraded to netstandard. #763

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Upgraded to netstandard. #763

wants to merge 1 commit into from

Conversation

guylando
Copy link

No description provided.

@guylando guylando mentioned this pull request May 17, 2018
@seriouz
Copy link

seriouz commented May 17, 2018

Would love to see this getting into the main repo!

@@ -0,0 +1,54 @@
<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="Build" ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why are you adding these files with the .old extension? just delete the old stuff

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well I have performed the first conversion and asked people to check if it works well. If it will not work we will see exactly what is missing in the "old" files. The "old" files have the csproj items which were not transferred to the new csproj. As soon as the library is verified we can delete the "old" files.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, please ask someone to test it, and if it works well, remove the .old files. Then after that we can merge this PR.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@seriouz can you test it?

@LORDofDOOM
Copy link

LORDofDOOM commented May 25, 2018

I've already added a proper PR some month ago: #687

As we already discuss in my PR upgrade to 4.6.1 is needed to respect coding rules for netcore/netstandard (net 4.5 is not compatible). Without 4.6.1 it could break on any library upgrade or is not working as library in other projects where coding follows netstandard rules.

Also BigMath and Ionic.Zlib are no netstandard compatible versions (PCL is deprecated by Microsoft for netstandard), for both are netstandard Version available - Please change them.

@ramtinak
Copy link

@LORDofDOOM for bigmath I've ported to NetStandard a long time ago:
https://www.nuget.org/packages/BigMathNetStandard/

and there is an Ionic.Zlib nestandard port too:
https://www.nuget.org/packages/Iconic.Zlib.Netstandard/

@LORDofDOOM
Copy link

LORDofDOOM commented Jun 10, 2018

@ramtinak: I know that, they are in my PR (#687) - I've wrote in this PR the versions are wrong (no netstandard)

@ilyalatt
Copy link

I created a fork with netstandard and published it via nuget, it works in my project.

@knocte
Copy link
Collaborator

knocte commented Dec 22, 2018

@ilyalatt can you create a PullRequest please?

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.

None yet

6 participants