Task #9253 removed logger envs witch have an autoconfiguration

[SaenkoVova]

GreenCity PR

Issue Link 📋

#9253

Summary by CodeRabbit

• Chores
  • Simplified logging configuration by removing unused settings to streamline application properties.

[coderabbitai]

Walkthrough

Logging configuration properties were removed from the application.properties file, including root logging level, handler-specific logging level, file path, file name, and file logging pattern. No new properties were introduced.

Changes

Cohort / File(s)	Change Summary
Logging Configuration Cleanup core/src/main/resources/application.properties	Removed five logging-related properties: logging.level.root, logging.level.greencity.exception.handler.CustomExceptionHandler, logging.file.path, logging.file.name, and logging.pattern.file

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• This change is straightforward configuration removal with no functional logic affected
• Verify that the removed logging properties are not required by other configuration files or relied upon in documentation

Poem

🧹 Log files fade from view,
Properties cleaned, configuration renewed,
Silence whispers clean and true,
Less noise, more focus—the old bid adieu! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The PR title "Task #9253 removed logger envs witch have an autoconfiguration" does relate to the main change in the changeset, which involves removing five logging-related properties from application.properties. The title accurately identifies that logging configurations with autoconfiguration capabilities were removed, capturing the essence of the change. However, the title contains a typo ("witch" should be "which") and uses somewhat vague terminology ("envs" is ambiguous—it could refer to environment variables, but the actual change is about removing properties from the application configuration file). Despite these presentation issues, the title remains comprehensible and directly relevant to the work performed.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch task/9253/Create-a-single-point-of-configuration-for-the-logging-system

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4829381 and 8a1ee8e.

📒 Files selected for processing (1)

• core/src/main/resources/application.properties (0 hunks)

💤 Files with no reviewable changes (1)

• core/src/main/resources/application.properties

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud