Skip to content

docs: add header comment for replace_logger script - #1570

Merged
ogazboiz merged 2 commits into
LabsCrypt:mainfrom
RohitMandal-cs:1507
Aug 3, 2026
Merged

docs: add header comment for replace_logger script#1570
ogazboiz merged 2 commits into
LabsCrypt:mainfrom
RohitMandal-cs:1507

Conversation

@RohitMandal-cs

@RohitMandal-cs RohitMandal-cs commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

##Summary

  • Added a header comment explaining the purpose of backend/replace_logger.cjs.
  • Documented that it is a one-time migration script and not part of the application runtime.

Fixes #1507

Pull Request Checklist

Please ensure your PR follows these steps, mirroring our CONTRIBUTING.md guidelines.

  • I have read the CONTRIBUTING.md document.
  • My code follows the code style of this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the documentation accordingly.
  • I have verified the changes locally.

@ogazboiz ogazboiz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

clear, truthful header comment, verified it matches what replace_logger.cjs actually does (regex logger call rewrite over a hardcoded file list, not imported at runtime). merging.

if you want to keep contributing, join us on Telegram: https://t.me/+DOylgFv1jyJlNzM0

@ogazboiz
ogazboiz merged commit 10a688d into LabsCrypt:main Aug 3, 2026
14 checks passed
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.

[Backend] backend/replace_logger.cjs is committed at the package root with no explanation

2 participants