You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucenenet.apache.org by GitBox <gi...@apache.org> on 2022/10/31 22:53:05 UTC
[GitHub] [lucenenet] NightOwl888 commented on a diff in pull request #735: Removed .NET Core 3.1 support.
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: dev-unsubscribe@lucenenet.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org