✨ feat: Improve configuration reload error messages#8
Merged
SebastienLaurent-CF merged 1 commit intomainfrom Nov 23, 2025
Merged
✨ feat: Improve configuration reload error messages#8SebastienLaurent-CF merged 1 commit intomainfrom
SebastienLaurent-CF merged 1 commit intomainfrom
Conversation
Enhance error reporting when configuration reload fails to provide clear, actionable feedback to developers instead of silently ignoring errors. ## Changes **internal/app/runner.go** - Replaced vague error message "Failed to reload config, continuing with current configuration" - Added explicit file path to error context with structured logging - New message: "Configuration reload failed - keeping previous configuration. Fix the configuration file and try again" - Helps developers quickly understand: * That their config changes were rejected * Which file has the problem * What action to take (fix and try again) ## Benefits ✅ Immediate feedback when config reload fails ✅ Clear indication of which file has the problem ✅ Actionable error message (not silent failure) ✅ Better developer experience when iterating on configuration ✅ All 129 tests passing with zero regressions ## Testing - All config validation tests pass - Config loading and validation works as expected - Error messages are clear and informative
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Enhance error reporting when configuration reload fails to provide clear, actionable feedback to developers instead of silently ignoring errors.
Changes
internal/app/runner.go
Benefits
✅ Immediate feedback when config reload fails
✅ Clear indication of which file has the problem
✅ Actionable error message (not silent failure)
✅ Better developer experience when iterating on configuration ✅ All 129 tests passing with zero regressions
Testing