[GitHub] [maven-indexer] carlspring commented on pull request #178: [MINDEXER-140] Fix double invocation of DefaultIndexingContext.setIndexDirectoryFile

[GitBox <gi...@apache.org>]

carlspring commented on pull request #178:
URL: https://github.com/apache/maven-indexer/pull/178#issuecomment-1038550746


   @cstamas : Thanks for the clarification! I believe thie was introduced due to my misunderstanding of how it should work.
   
   It would really help if there is a detailed page with examples on how to use the Maven Indexer. It's great to have example code, but if you'd like more people to understand this, there ought to also be some more thorough documentation. As this project has not been touched in many years and since there have been quite a few changes, I think that now would be a great time to add such documentation. It would also be great, if it could come from someone like yourself, who has worked a long time on this project and knows all the intricacies and peculiarities.
   
   On a different topic, it would probably also be great, if any remaing code that refers to Nexus could be removed in order to avoid confusion, (if I recall correctly, there were classes such as `NexusIndexer`, etc., forgive me, I have not dug around the code in years). As far as I recall, this is a generic implementation that is unrelated to Nexus itself, right?
   
   Thanks for your hard work on pushing the `maven-indexer` forward! :)
   


-- 
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: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org