Skip to content

Conversation

@And1sS
Copy link
Collaborator

@And1sS And1sS commented Oct 23, 2025

🔧 Type of changes

  • new bid adapter
  • bid adapter update
  • new feature
  • new analytics adapter
  • new module
  • module update
  • bugfix
  • documentation
  • configuration
  • dependency update
  • tech debt (test coverage, refactorings, etc.)

✨ What's the context?

Logs can clutter storage, so we decided to change log level from info to debug.

🧠 Rationale behind the change

Now log level is lower - so logs will appear only when needed.

🧪 Test plan

Logging in this case is not a functional requirement, so it is not tested.

🏎 Quality check

  • Are your changes following our code style guidelines?
  • Are there any breaking changes in your code?
  • Does your test coverage exceed 90%?
  • Are there any erroneous console logs, debuggers or leftover code in your changes?

@And1sS And1sS requested a review from CTMBNara October 23, 2025 12:53
@And1sS And1sS changed the title Changed level for rule parsing log. Rule Engine: Change level for rule parsing log Oct 23, 2025
@CTMBNara CTMBNara merged commit 9976884 into master Oct 27, 2025
8 checks passed
@CTMBNara CTMBNara deleted the rule-engine/parser-logging-update branch October 27, 2025 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants