[GitHub] [lucenenet] laimis opened a new pull request, #822: Ignore sonar cloud errors for DocTermOrds

["laimis (via GitHub)" <gi...@apache.org>]

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

   Continuation of fixes with virtual calls being made from constructors. The issue originally reported by SonarCloud scans: https://sonarcloud.io/project/issues?resolved=false&rules=csharpsquid%3AS1699&id=apache_lucenenet and referenced in this issue: https://github.com/apache/lucenenet/issues/670
   
   We are ignoring DocTermOrds errors as the Lucene team intended for the subclasses to use the protected constructor that does not call the virtual method, and they even specify that subclasses should call the virtual method themselves.


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

[GitHub] [lucenenet] laimis merged pull request #822: Ignore sonar cloud errors for DocTermOrds

["laimis (via GitHub)" <gi...@apache.org>]

laimis merged PR #822:
URL: https://github.com/apache/lucenenet/pull/822


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