WIP: feat: support for NET Core 2.0#95
Conversation
| </ProjectReference> | ||
| </ItemGroup> | ||
| <ItemGroup> | ||
| <None Include="test-keys\cert.pem" /> |
There was a problem hiding this comment.
<None Include="test-keys\cert.pem" /> was moved up in the .csproj file. This is a TLS example project.
|
|
||
| namespace NsqSharp.WindowService | ||
| { | ||
| public class WindowsBusConfiguration : BusConfiguration, IWindowsBusConfiguration |
There was a problem hiding this comment.
Easiest way to implement IsConsoleMode
| static InterfaceBuilder() | ||
| { | ||
| var assemblyName = new AssemblyName(Guid.NewGuid().ToString()); | ||
| #if NETFULL |
There was a problem hiding this comment.
Target dependent implementation
| } | ||
|
|
||
| return typeBuilder.CreateType(); | ||
| return typeBuilder.CreateTypeInfo(); |
There was a problem hiding this comment.
New method supported in both platforms
| } | ||
|
|
||
| [TestFixtureTearDown] | ||
| [OneTimeTearDown] |
There was a problem hiding this comment.
TestFixtureTearDown is no deprecated in favor of OneTimeTearDown
| preCreateTopicsAndChannels: true); | ||
|
|
||
| BusService.Start(busConfiguration); | ||
| busConfiguration.StartBus(); |
There was a problem hiding this comment.
All these tests require this change now. BusService is not supported in NET Core/Standard
|
Thanks @alex-pollan, I agree .NET Standard 2.0 is the way to go. Since the Windows Service code will be pulled into another package now is a good time to look again at how that works. I'll do a major version bump of NsqSharp to account for the breaking change. |
|
@alex-pollan I'm also okay dropping support for any .NET less than 4.6.1 per the compatibility matrix https://docs.microsoft.com/en-us/dotnet/standard/net-standard. Users needing earlier .NET support can use 0.5.4 which has been battle tested in Production for quite some time. |
alex-pollan
force-pushed
the
feat/netcore
branch
from
87f1719 to
7d86ff0
Compare
Done, now it only supports 4.6.1+ |
|
@judwhite addressed your comments |
|
@alex-pollan @judwhite any updates on this? |
3 participants
NsqSharptargets NET Framework4.54.6.1+ and NET Standard 2.0NsqSharp.WindowServicetargeting NET4.5.14.6.1 to keep theBusServicestatic class with support for Window Service, console app and host in IISNsqSharp(net4.54.6.1)To do:
NsqSharp.WindowService?