Breaking: EntityListFile changes to EntityListFiles - #28
Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a breaking configuration change for the analyzer: replacing the single entityListFile input with plural entityListFiles to support specifying multiple entity list files, and updates tests/docs accordingly.
Changes:
- Rename config/CLI flag from
entityListFiletoentityListFilesand switch internal state from a single string to a slice of paths. - Update analyzer initialization to load and union protected entities across multiple files.
- Update tests and README to reflect the new multi-file configuration and CLI usage.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| pkg/analyzer/analyzer.go | Implements entityListFiles parsing from config/CLI and unions entity lists across files. |
| pkg/analyzer/analyzer_test.go | Updates tests for the new config key and adds a multi-file test case. |
| README.md | Documents the new YAML/CLI configuration for multiple entity list files. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
This PR updates the analyzer configuration to accept multiple “entity list” files instead of a single file, and updates tests/documentation accordingly.
Changes:
- Rename config/flag from
entityListFiletoentityListFilesand switch parsing to a multi-file list. - Update analyzer initialization and map-building logic to union protected structs across multiple files.
- Expand test coverage and update README examples/CLI documentation for the new parameter.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| pkg/analyzer/analyzer.go | Switches config/flag parsing to multi-file input and unions protected structs across files. |
| pkg/analyzer/analyzer_test.go | Updates existing tests and adds new cases for multi-file input and config decoding ([]any). |
| README.md | Documents the new entity-list-files YAML config and -entityListFiles CLI flag usage. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Input parameter can now receive multiple files listing entities instead of one.