Christian Stewart
Christian Stewart
This was merged but it may still require some follow-up cleanup https://github.com/IrcDotNet/IrcDotNet/pull/51
That's fine. Thanks for PRs!
Newer features are fine, as long as we can produce binaries that are somewhat compatible with older .net versions.
You're right - shouldn't be on nuget. I'll look at fixing that.
I think the majority of the debug assert statements fail when running against a nonstandard server, by the way, and expected behaviors aren't happening.
Debug assertions should NOT be enabled at release. Marking as a bug and renaming as such.
Should be fixed tomorrow with the release of a dotnetstandard version.
Those assertions are there to probe for bugs in the testing phase. They are not included in the release binaries anymore.
@chrboe others are merged now
Have you tried manually sending a disconnect message before sending quit()?