-
Notifications
You must be signed in to change notification settings - Fork 11
[WIP] .net core #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
[WIP] .net core #16
Conversation
| <Project Sdk="Microsoft.NET.Sdk"> | ||
|
|
||
| <PropertyGroup> | ||
| <TargetFramework>netcoreapp1.1</TargetFramework> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In order to add .net45 support you need to write: <TargetFramework>netcoreapp1.1;net45</TargetFramework>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could actually use https://docs.microsoft.com/en-us/dotnet/standard/net-standard netstandard1.1 to support .net 45 and core ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for the tests I'm not sure that there is a need for multiple target frameworks
|
Is this going to be merged? I really need .Net Core support to use Assembly. |
|
First they need to review the changes. It's fully possible that this quite large pull request will be rejected. I've done a lot of changes that they might not agree with. For instance I've tried to refactor it a bit to match more the style of .net core libraries. I've removed the "Dynamic" parts (in order to be a bit more .net-y and cut down on the amount of code that needs to be tested). What you can do is to create a public nuget feed and do |
|
Instead of targeting .net 45 targeting netstandard: https://docs.microsoft.com/en-us/dotnet/standard/net-standard |
|
We have made a carable fork of it: https://github.com/carable/carable.assemblypayments |
Uh oh!
There was an error while loading. Please reload this page.