๐Ÿ› Fix: Prevent nil errors in log.Error to ensure proper logging and add custom linter to avoid this scenario in the future by camilamacedo86 ยท Pull Request #1599 ยท operator-framework/operator-controller

@camilamacedo86 camilamacedo86 changed the title ๐Ÿ› Fix error message ๐Ÿ› Fix: Prevent nil errors in setupLog.Error to ensure proper logging and add sanity check

Jan 10, 2025

joelanford

@camilamacedo86 camilamacedo86 changed the title ๐Ÿ› Fix: Prevent nil errors in setupLog.Error to ensure proper logging and add sanity check (WIP) ๐Ÿ› Fix: Prevent nil errors in setupLog.Error to ensure proper logging and add sanity check

Jan 12, 2025

joelanford

joelanford

joelanford

joelanford

@camilamacedo86 camilamacedo86 changed the title (WIP) ๐Ÿ› Fix: Prevent nil errors in setupLog.Error to ensure proper logging and add sanity check ๐Ÿ› Fix: Prevent nil errors in setupLog.Error to ensure proper logging and add sanity check

Feb 5, 2025

tmshort

tmshort

tmshort

tmshort

tmshort

tmshort

tmshort

@camilamacedo86

โ€ฆ.) calls and fix found linter issues.

Fix: Prevent nil errors in setupLog.Error to ensure proper logging

Closes; operator-framework#1566
Closes: operator-framework#1556

Furthermore, it solves a similar scenario in the EventHandler code implementation found with the new custom linter check.

Merged via the queue into operator-framework:main with commit 4304961

Feb 11, 2025

20 of 22 checks passed