rclabo commented on code in PR #963: URL: https://github.com/apache/lucenenet/pull/963#discussion_r1777007465
########## src/Lucene.Net.Analysis.Stempel/Egothor.Stemmer/Compile.cs: ########## @@ -74,6 +81,13 @@ public static class Compile // LUCENENET specific: CA1052 Static holder types sh private static Trie trie; /// <summary> + /// <para> + /// LUCENENET specific: This class is not for direct use. In the Java implementation + /// it's Main method was intended to be called from the command line. However in .NET a + /// method within a DLL can't be directly called from the command line so we + /// provide a <see href="https://www.nuget.org/packages/lucene-cli">lucene-cli</see> + /// with a command that maps to that method: analysis stempel-compile-stems. + /// </para> Review Comment: Yah, I went back and forth about that. but I like the <para>text </para> approach better in cases where a chunk of the text is luceneNET specific. -- 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