Skip to content
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

Update to net45 targetframework #1033

Closed
wants to merge 3 commits into from
Closed

Conversation

Titaye
Copy link

@Titaye Titaye commented Jan 17, 2018

I'm planning to include ICSharpCode.Decompiler nuget package to https://github.com/fsharp/FsAutoComplete which is targeting .net 4.5. Can you release a new version of your libs targeting this version ? The only compatibility issue is the use of Array.emtpy

@christophwille
Copy link
Member

We have switched fw's specifically for Roslyn integration. We are not planning on changing this targetting.

@christophwille
Copy link
Member

Why didn't you go with netstandard2.0;net46;net45? three tfm, three dlls in the nupkg.

@Titaye
Copy link
Author

Titaye commented Jan 17, 2018

Yes of course no pb for having the 3 targetting. For compatibility reasons, I'm don't think FSAC will be able to move to net46. If you don't mind adding net45 to the existings list, it will be perferct.

@siegfriedpammer
Copy link
Member

I have committed a slightly modified version of your changes in 1e14c8f. Not sure, when we will be able to publish a beta for 3.1 on nuget.org. In the meantime you can use our private nuget feed.

@Titaye
Copy link
Author

Titaye commented Jan 18, 2018

this is perfect. thanks.

siegfriedpammer added a commit that referenced this pull request Jan 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants