search

jitbit / AspNetSaml

Very simple SAML 2.0 consumer module for ASP.NET/C#
https://www.jitbit.com
Apache License 2.0
361 stars 118 forks source link

Don't install by copying a file into the consuming project #73

Closed atrauzzi closed 1 year ago

atrauzzi commented 1 year ago

The proper convention for doing a nuget packages is to compile it and then have those files be referenced. Not by injecting a file into the consuming project.

This is particularly bad if for any reason your source file and/or its conventions do not 100% agree with the project you're forcing it into.

It also gives less experienced users a sense that they can modify the file, when really they can't and shouldn't (otherwise it defeats the purpose of having a package).

image

alex-jitbit commented 1 year ago

Fair point, this IS already in the works, actually. Hope to release this week.

alex-jitbit commented 1 year ago

https://www.nuget.org/packages/AspNetSaml/2.0.0-beta (pre release)