Skip to content

Conversation

@xdustinface
Copy link
Collaborator

@xdustinface xdustinface commented Jan 29, 2026

Enable module path (target) in log output to allow filtering by module.

before:

2026-01-29T01:30:02.487715+0100 INFO  Starting sync...
2026-01-29T01:30:02.487715+0100 DEBUG Processing block 12345

now:

2026-01-29T01:30:02.487715+0100 INFO  dash_spv::sync::manager Starting sync...
2026-01-29T01:30:02.487715+0100 DEBUG dash_spv::storage::block_storage Processing block 12345

Summary by CodeRabbit

Release Notes

  • Bug Fixes
    • Console logs now display target names alongside messages, providing improved visibility of logging context.

✏️ Tip: You can customize this high-level summary in your review settings.

Enable module path (target) in log output to allow filtering by module.

before:
```bash
2026-01-29T01:30:02.487715+0100 INFO  Starting sync...
2026-01-29T01:30:02.487715+0100 DEBUG Processing block 12345
```
now:
```bash
2026-01-29T01:30:02.487715+0100 INFO  dash_spv::sync::manager Starting sync...
2026-01-29T01:30:02.487715+0100 DEBUG dash_spv::storage::block_storage Processing block 12345
```
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 29, 2026

📝 Walkthrough

Walkthrough

The console logging layer now includes log record target names in output by setting with_target(true) instead of with_target(false). This single-line change enriches console log messages with target name information.

Changes

Cohort / File(s) Change Summary
Logging Configuration
dash-spv/src/logging.rs
Enabled target name inclusion in console log output by changing with_target(false) to with_target(true)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A single flag now set to true,
The logs reveal their source anew,
With targets bright in each log line,
Our bunny's debugging shines divine! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title 'feat: include module names in the log output' accurately describes the main change: enabling module paths (log targets) in console logging by changing with_target from false to true.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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

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

@xdustinface xdustinface merged commit 9323261 into v0.42-dev Jan 29, 2026
53 checks passed
@xdustinface xdustinface deleted the feat/log-target branch January 29, 2026 15:51
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