Skip to content

refactor: add debug_log and use it replacing guarded logs#69

Open
snawaz wants to merge 2 commits intographite-base/69from
snawaz/debug-log
Open

refactor: add debug_log and use it replacing guarded logs#69
snawaz wants to merge 2 commits intographite-base/69from
snawaz/debug-log

Conversation

@snawaz
Copy link
Copy Markdown
Contributor

@snawaz snawaz commented Apr 11, 2026

  • added a friendly debug_log! which is enabled with feature = "logging".

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 11, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 8ba910e0-7cc7-4bca-89ba-504593ed0c81

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch snawaz/debug-log

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.

Copy link
Copy Markdown
Contributor Author

snawaz commented Apr 11, 2026

@snawaz snawaz force-pushed the snawaz/debug-log branch from 2a7f5d3 to a50de0a Compare April 11, 2026 19:36
@snawaz snawaz force-pushed the snawaz/debug-log branch 2 times, most recently from e51e72f to dfb67d2 Compare April 12, 2026 01:12
@snawaz snawaz force-pushed the snawaz/heap branch 2 times, most recently from 4a11113 to 6f21915 Compare April 12, 2026 08:44
@snawaz snawaz force-pushed the snawaz/debug-log branch 2 times, most recently from 25e2c48 to e470c26 Compare April 12, 2026 13:05
@snawaz snawaz force-pushed the snawaz/heap branch 2 times, most recently from 855bce5 to c58b3bb Compare April 12, 2026 20:49
@snawaz snawaz force-pushed the snawaz/debug-log branch from e470c26 to 9afdce9 Compare April 12, 2026 20:49
@snawaz snawaz marked this pull request as ready for review April 12, 2026 21:15
@snawaz snawaz force-pushed the snawaz/debug-log branch 2 times, most recently from 4be28dc to 814868b Compare April 13, 2026 11:20
@snawaz snawaz requested a review from GabrielePicco April 13, 2026 11:22
@snawaz snawaz force-pushed the snawaz/debug-log branch from 814868b to 0c34e82 Compare April 13, 2026 15:33
Copy link
Copy Markdown
Contributor

@GabrielePicco GabrielePicco left a comment

Choose a reason for hiding this comment

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

LGTM!

@snawaz snawaz force-pushed the snawaz/debug-log branch from 0c34e82 to 783898a Compare April 14, 2026 10:48
Copy link
Copy Markdown
Contributor

@Dodecahedr0x Dodecahedr0x left a comment

Choose a reason for hiding this comment

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

LGTM

@snawaz snawaz changed the base branch from snawaz/heap to graphite-base/69 April 14, 2026 15:52
@snawaz snawaz force-pushed the graphite-base/69 branch from 6304f71 to b0f26b7 Compare April 14, 2026 15:53
@snawaz snawaz force-pushed the snawaz/debug-log branch 2 times, most recently from 3fd87e4 to d07f5e7 Compare April 14, 2026 16:38
@snawaz snawaz force-pushed the graphite-base/69 branch 2 times, most recently from 484772c to 8dec4d6 Compare April 14, 2026 17:20
@snawaz snawaz force-pushed the snawaz/debug-log branch 2 times, most recently from 075c5d7 to 1dc3eda Compare April 14, 2026 18:15
@snawaz snawaz force-pushed the graphite-base/69 branch from 8dec4d6 to 2b36173 Compare April 14, 2026 18:15
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