NightOwl888 commented on code in PR #735:
URL: https://github.com/apache/lucenenet/pull/735#discussion_r1009934152
##########
TestTargetFramework.props:
##########
@@ -28,14 +28,13 @@
this setting only affects the test projects. -->
<!--<TargetFramework>net461</TargetFramework>-->
<!--<TargetFramework>net48</TargetFramework>-->
- <!--<TargetFramework>netcoreapp3.1</TargetFramework>-->
<TargetFramework>net5.0</TargetFramework>
<!--<TargetFramework>net6.0</TargetFramework>-->
<!-- Allow the build script to pass in the test frameworks to build for.
This overrides the above TargetFramework setting.
LUCENENET TODO: Due to a parsing bug, we cannot pass a string with a ;
to dotnet msbuild, so passing true as a workaround -->
- <TargetFrameworks Condition=" '$(TestFrameworks)' == 'true'
">net6.0;net5.0;netcoreapp3.1</TargetFrameworks>
+ <TargetFrameworks Condition=" '$(TestFrameworks)' == 'true'
">net6.0;net5.0;</TargetFrameworks>
Review Comment:
There is an extra `;` at the end here that can be removed. When this value
is concatenated with the values below, we end up with an empty array element
`;;`.
##########
src/dotnet/tools/lucene-cli/docs/index.md:
##########
@@ -1,10 +1,10 @@
-# Lucene.NET command line interface (CLI) tools
+# Lucene.NET command line interface (CLI) tools
The Lucene.NET command line interface (CLI) is a new cross-platform toolchain
with utilities for maintaining Lucene.NET and demos for learning basic
Lucene.NET functionality.
## Prerequisites
-- [.NET Core 3.1 Runtime or
Higher](https://dotnet.microsoft.com/en-us/download/dotnet)
+- [.NET Core 5.0 Runtime or
Higher](https://dotnet.microsoft.com/en-us/download/dotnet)
Review Comment:
Technically, it is `.NET 5.0 Runtime or Higher`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
