Claudenw commented on PR #433: URL: https://github.com/apache/creadur-rat/pull/433#issuecomment-2640552627
@raboof Are some of the things you put in your private ignore file found in other StandardCollection entries? Would it make sense to create more StandardCollection entries for anything that is missing? The changes that you want to work against are in Pull request #439 Specifically: - in the `AbstractFileProcessorBuilder.build(final DocumentName root)` you want to add a call to a new method `void initializeBuilder(final DocumentName root)` that you will create in `AbstractFileProcessorBuilder` as a method that does nothing. - in `GitIgnoreBuilder` you want to implement `initializeBuilder()` to perform the file name lookup logic and if a file is found process it using by createing a `LevelBuilder` for level `-1` and calling `protected MatcherSet process(final Consumer<MatcherSet> matcherSetConsumer, final DocumentName root, final DocumentName documentName)` to process the file. See `AbstractFileProcessBuilder.checkDirectory()` for an example of creating and using the LevelBuilder. Bascically the system works by creating `MatcherSet`s for each level (.gitignore) file and then applies them in reverse order so that the later ones override the earlier ones as is requried by the gitignore spec. A `MatcherSet` has two matchers, one that specifies the files that are to be _explicitly_ included (overrides earlier excludes), and one to list files to exclude. The `process()` method will do that correctly if you have the proper `root` and `documentName` specified. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
