search

adams85 / bundling

A library for optimizing and bundling web assets of ASP.NET Core applications.
MIT License
42 stars 7 forks source link

Make non tooling projects netstandard2.0 #5

Closed schotime closed 5 years ago

schotime commented 5 years ago

Great project firstly. Do you have any objections to making the projects netstandard2.0 so they can be referenced by other netstandard2.0 projects. I wanted to abstract some logic in this library but was unable to do so.

adams85 commented 5 years ago

Thanks, Adam!

Sure, we can do this. Even the ASP.NET Core libs referenced in the main project (Microsoft.AspNetCore.Mvc.TagHelpers and Microsoft.AspNetCore.StaticFiles) are netstandard2.0. However, switching the project to netstandard would make perfect sense for me if we also refactored the ASP.NET Core agnostic stuff into a separate core assembly. I need to give this some more thought.

By the way, what's your reason for changing StartsWith/EndsWith/etc overloads to use strings instead of chars?

adams85 commented 5 years ago

Ok, I think I've got it. Those overloads don't exist in netstandard2.0, do they? :)

schotime commented 5 years ago

Hi, yeah those overloads don't exist.

I've successfully integrated my branch into our project with the netstandard2.0 changes as is and everything seems to work as is.

My shared library is just referencing the bundling core library and works great.

Separating the asp.net stuff could happen but I'm ok with how it is.

schotime commented 5 years ago

Any progress on this? I'd rather not run my own fork ;)

adams85 commented 5 years ago

Sorry, I've been busy recently.

I needed to do some quick tests to see if this change won't break the dotnet CLI tool. It seems it's alright so I'll merge your PR and publish a new version shortly.

schotime commented 5 years ago

All good mate, I know what it's like. Thanks.