novuhq / novu-dotnet

.NET SDK for Novu - The open-source notification infrastructure for engineers. 🚀
MIT License
26 stars 13 forks source link

Release Extensions and Sync dlls #55

Closed toddb closed 1 year ago

toddb commented 1 year ago

Novu has three parts/dlls:

There is a decision:

The code as part of 0.3.0 did not release this code.

Note: Novu may need to be called Novu.Core (or the like) indicating base.

toddb commented 1 year ago

My vote is to take the hit now and release them separately allowing for people to have as few a dependencies to allow for those that have different environments and minimal requirements.

However, another way to merge Extensions back into Core and only release two packages. But Core then has dependencies on Microsoft around DI and Configuration—any business software will have this (IMHO). Much of the extensions code are utility helpers that live as extension methods which are not technically extensions but extension methods.

wh1337 commented 1 year ago

I too agree that we change the packaing to the following:

If we take the hit now, it will be better going forward.

@unicodeveloper thoughts?

wh1337 commented 1 year ago

I'll take this one and get it out in #61

toddb commented 1 year ago

Released 0.3.1