fix(registry): guard against nil logger in RelationshipCSVHelper - #1086
fix(registry): guard against nil logger in RelationshipCSVHelper#1086Omkar-Ugal wants to merge 1 commit into
Conversation
Signed-off-by: Omkar Ugalmugle <omkarugalmugle285@gmail.com>
|
Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, you can find updates in the #github-notifications channel in the community Slack. |
📝 WalkthroughWalkthroughRelationship parsing now guards informational and error logging with nil checks. Parsing errors remain returned to the caller. ChangesRelationship parsing
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
This fixes a nil pointer panic in
RelationshipCSVHelper.ParseRelationshipsSheet()whenLogis not initialized.Signed commits
Summary by CodeRabbit