search

ljnath / RandomString4Net

.NET library to generate N random strings of M length from various categories
MIT License
9 stars 2 forks source link

Updated random seed to use 4 bytes rather than just 1 #4

Open jshergal opened 1 month ago

jshergal commented 1 month ago

This resolves #2

ljnath commented 1 month ago

Thank you for the PR, this was long pending from my end.

jshergal commented 1 month ago

@ljnath The build is failing because the tooling is not installed for all of the targeted frameworks. I am making an update to my other PR that you rejected due to desiring to support older frameworks. I found that NUnit still supports testing .NET Framework 3.5, so I am adding a unit test project in NUnit to test .NET Framework 3.5+. I am adding the target back in for .NET Framework 2.0, and then keeping standard 2.0 and 2.1. Anyway, If you accept that PR, then I can go ahead and pull those changes into this PR and make the update. Assuming the other PR builds without issue, that should address this build failure.