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

Add support for DNX/.NET Core #59

Closed
glen-84 opened this issue Jun 8, 2015 · 20 comments
Closed

Add support for DNX/.NET Core #59

glen-84 opened this issue Jun 8, 2015 · 20 comments
Labels

Comments

@glen-84
Copy link

glen-84 commented Jun 8, 2015

No description provided.

@SeriousM
Copy link
Contributor

SeriousM commented Jun 8, 2015

you're invited to contribute.

btw, a one-liner, even it's a good idea, is kind of unpleasant. we're providing this great piece of software for free; treat it with love.

@glen-84
Copy link
Author

glen-84 commented Jun 8, 2015

I apologize, I didn't think that a description was necessary. I will check what needs to be done in order to support DNX and .NET core.

@SeriousM
Copy link
Contributor

SeriousM commented Jun 8, 2015

That would be great, thank you!
Am 08.06.2015 17:13 schrieb "Glen" [email protected]:

I apologize, I didn't think that a description was necessary. I will check
what needs to be done in order to support DNX and .NET core.


Reply to this email directly or view it on GitHub
#59 (comment)
.

@glen-84
Copy link
Author

glen-84 commented Jun 8, 2015

I really don't know how to retrofit this and maintain the existing project and solution files.

The best I could do is to implement this on another branch, without all the old project/solution files, etc.

@SeriousM
Copy link
Contributor

I don't know what's the benefit of supporting DNX and what do you mean with .NET Core.
Could you just introduce us to the wished changes so that we can understand it without starting with the research from scratch?
Thanks

@glen-84
Copy link
Author

glen-84 commented Jun 10, 2015

.NET Core & DNX.

I'm going to try to implement this on a clean branch.

@glen-84
Copy link
Author

glen-84 commented Jun 11, 2015

https://github.com/glen-84/SmartFormat.NET/tree/DNX

It's building for dnx451 (regular framework under DNX), but there are 13 errors for DNX Core (~7 unique). You'll need to go through the list and find out if there are plans to add support for certain types/methods in .NET Core, and if not, find alternatives (assuming that you do want to support .NET Core).

Removed:

  1. "releases" -- this is not something that's usually stored inside a source repo.
  2. SmartFormat.Demo -- not sure if it's possible to run a WinForms app with DNX.
  3. Extra solution files, NuGet stuff, etc. -- some things can be added back if necessary.

Notes:

  1. NUnit does not yet support .NET Core (issue). You can either wait or switch to something like xUnit.

@SeriousM
Copy link
Contributor

@glen-84, when do you expect that this is supported by SmartFormat.NET? Is it something you really wait for it or do you just want to support the project?

NUnit does not yet support .NET Core (issue). You can either wait or switch to something like xUnit.

I guess we will wait :)

@glen-84
Copy link
Author

glen-84 commented Jun 11, 2015

I was writing an implementation of an IStringLocalizer for ASP.NET 5 that allows the developer to use a custom string provider and string formatter, and I was going to use SmartFormat.NET and MessageFormat.NET as examples for the formatter.

i.e. It's not super-high priority.

@ghost
Copy link

ghost commented Jul 21, 2015

Any updates on this? Microsoft just released the next version of Visual Studio with .NET CORE and we would like to migrate our software to the new .NET version.

@axunonb
Copy link
Member

axunonb commented Oct 18, 2016

Created a Pull Request for supporting .NET Core (Standard 1.6). Finally supporting Async in both: .Net Framwork 2-4, but also .NET Core.
Any comments or discussion welcome.

@axunonb
Copy link
Member

axunonb commented Oct 23, 2016

@scottrippey, @SeriousM: Priorities and interest in projects change, that's absolutely fine for me. Since MailMergeLib relies on SmartFormat.Net I would just like to know, whether there will be further development of this GitHub repository, or whether I'm on my own.

@scottrippey
Copy link
Contributor

@axunonb Norbert, sorry for the delayed responses. As you've correctly guessed, I'm unable to continue development of this repo, as I've been out of .NET development for many years.
I'd be happy to add you as a collaborator, if you're interested. I'm not sure how NuGet publishing works, but I'd be willing to share my keys or transfer ownership, whatever's necessary!
Sorry about the lack of activity, I'm glad you found this package useful!

@axunonb
Copy link
Member

axunonb commented Oct 26, 2016

@scottrippey Scott, I'm sorry to hear that, and I also think it would be a pity to stop working on SmartFormat.Net. So yes, you may add me as a collaborator and I'll try my best. With NuGet publishing I'll have to find out what to do. I'll let you know. For now thanks again for the good work with this repo.

@axunonb
Copy link
Member

axunonb commented Oct 26, 2016

@scottrippey Scott, I had just been checking various options on how to go on with SmartFormat.Net. So:

@SeriousM
Copy link
Contributor

SeriousM commented Oct 26, 2016

Adding you as a collaborator is the most simple thing because the URL stays
the same.

@scottrippey
Copy link
Contributor

@axunonb I've added you as both a GitHub contributor and a Nuget Owner! With great power comes great responsibility! :)
Thanks in advance for your help!

@axunonb
Copy link
Member

axunonb commented Oct 26, 2016

Okay I confirmed both, thanks. @SeriousM are you still actively contributing? I'd be interested in your opinion about this Pull Request.
I also confirmed these issues which would be nice to "disappear".

@axunonb
Copy link
Member

axunonb commented Nov 1, 2016

Pull request merged.

@axunonb axunonb closed this as completed Nov 1, 2016
@SeriousM
Copy link
Contributor

SeriousM commented Nov 1, 2016

Hi Norbert, I'm not an active contributor anymore.
Sorry Scott, but my life is pretty demanding.
Cheers!

Am 01.11.2016 10:17 schrieb "Norbert Bietsch" [email protected]:

Closed #59 #59.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#59 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcMvVb8AQROgTVjaIQvYikHhNNKcaMCks5q5wONgaJpZM4E7uMC
.

axunonb pushed a commit to axunonb/SmartFormat that referenced this issue Dec 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants