Add label, start using it in crawler#197
Conversation
Potential Risks and Suggestions:File: src/crawl/pom.py File: src/crawl/artifactgenctx.py File: src/crawl/crawler.py File: src/crawl/pom.py File: misc/extdeps_pomgen.py Summary:This PR introduces significant architectural changes to the dependency management system in the Salesforce pomgen tool. The main changes include:
The changes appear to be well-structured and include comprehensive test coverage for the new functionality. The PR primarily focuses on internal refactoring to improve code maintainability and readability, rather than changing external behavior. The most significant risk is in the transition from string-based label handling to the new Label class, but the extensive test coverage should mitigate this risk. The changes to method signatures (particularly removing the This refactoring appears to be a positive improvement to the codebase, making it more maintainable and less error-prone by providing proper abstractions for Bazel labels. The PR includes appropriate tests and seems ready for integration. |
No description provided.