Skip to content

update error levels#192

Merged
goodviber merged 1 commit intomainfrom
error-level
Apr 7, 2025
Merged

update error levels#192
goodviber merged 1 commit intomainfrom
error-level

Conversation

@goodviber
Copy link
Copy Markdown
Contributor

We are rescuing initialisation and other errors with Rails.logger.info messages, which makes them difficult to find In the output logs. This PR updates the relevant info messages to error or warn level

@goodviber goodviber added enhancement New feature or request Do not merge Not ready for merging labels Apr 3, 2025
@goodviber goodviber requested a review from asatwal April 3, 2025 15:44
@goodviber goodviber self-assigned this Apr 3, 2025
@goodviber goodviber marked this pull request as ready for review April 3, 2025 15:47
Copy link
Copy Markdown
Collaborator

@asatwal asatwal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM

@asatwal asatwal mentioned this pull request Apr 7, 2025
@goodviber goodviber removed the Do not merge Not ready for merging label Apr 7, 2025
@goodviber goodviber merged commit f5b0ca6 into main Apr 7, 2025
5 checks passed
@goodviber goodviber deleted the error-level branch April 7, 2025 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants