paulirwin opened a new pull request, #897:
URL: https://github.com/apache/lucenenet/pull/897

   - [X] You've read the [Contributor 
Guide](https://github.com/apache/lucenenet/blob/main/CONTRIBUTING.md) and [Code 
of Conduct](https://www.apache.org/foundation/policies/conduct.html).
   - [X] You've included unit or integration tests for your change, where 
applicable.
   - [X] You've included inline docs for your change, where applicable.
   - [X] There's an open issue for the PR that you are making. If you'd like to 
propose a change, please [open an 
issue](https://github.com/apache/lucenenet/issues/new/choose) to discuss the 
change or find an existing issue.
   
   Test review for the T-Z unit test project.
   
   Associated #259 (doesn't fully fix)
   
   ## Description
   
   - This addresses some formatting disparities between the Java and C# tests 
to help make them more line-to-line matching.
   - Comments added in places where our tests differ from upstream.
   - RandomizedTesting.Generators extension method import moved behind a `#if 
!NET6_0_OR_GREATER` check, as the namespace was being ignored on .NET 6+ in a 
few files since .NET 6 added a Random.NextInt64 method. This will help prevent 
other contributors from thinking this namespace is not needed.
   - A unit test assertion bug on arm64 with division by zero is fixed.
   - The `build` file has been chmod'ed +x for ease of use
   - Rider IDE files added
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to