You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucenenet.apache.org by "paulirwin (via GitHub)" <gi...@apache.org> on 2024/10/23 20:00:13 UTC

[PR] Update README.md for .NET support [lucenenet]

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

   - [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.
   - [ ] 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.
   
   Updates the .NET support information in the README.
   
   Related to #928 
   
   ## Description
   
   Updates the README to add the .NET 8 target, specify that the CLI requires the .NET 8 runtime, and removes references to older unsupported frameworks.
   


-- 
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


Re: [PR] Update README.md for .NET support [lucenenet]

Posted by "NightOwl888 (via GitHub)" <gi...@apache.org>.
NightOwl888 merged PR #992:
URL: https://github.com/apache/lucenenet/pull/992


-- 
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


Re: [PR] Update README.md for .NET support [lucenenet]

Posted by "NightOwl888 (via GitHub)" <gi...@apache.org>.
NightOwl888 commented on code in PR #992:
URL: https://github.com/apache/lucenenet/pull/992#discussion_r1813470393


##########
README.md:
##########
@@ -37,9 +38,9 @@ Working toward Lucene.NET 4.8.0 (currently in BETA)
 
 * The beta version is extremely stable
 * Has more than 7800+ passing unit tests
-* Integrates well with .NET 6.0, .NET 5.0 and .NET Core 2+
+* Integrates well with .NET 8.0 and .NET 6.0 (as well as other unsupported versions)

Review Comment:
   I was just reading a comment yesterday that I left about this. This tends to go out of date faster than we can update it. So, maybe we should just say something along the lines of "Tested daily on all [officially supported versions of .NET Core](https://dotnet.microsoft.com/en-us/platform/support/policy/dotnet-core)".
   
   The targets have a loose relationship with the runtimes, but I suspect anyone reading this just wants to know if their preferred runtime will work. If anyone wants to see our supported target frameworks, it is all on the NuGet package page. Maybe we should just drop a link to: https://www.nuget.org/packages/Lucene.Net/absoluteLatest#supportedframeworks-body-tab, which will always be up to date.
   
   Also, this information was copied over to our homepage, which will also need to be updated: https://lucenenet.apache.org/. At one point, these 2 descriptions were identical, but they don't remain in sync because they don't share the same source file.



-- 
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