Skip to content

fix: default error logs enabled - #1525

Merged
GHkrishna merged 2 commits into
mainfrom
feat/prisma-logging
Nov 24, 2025
Merged

fix: default error logs enabled#1525
GHkrishna merged 2 commits into
mainfrom
feat/prisma-logging

Conversation

@GHkrishna

@GHkrishna GHkrishna commented Nov 24, 2025

Copy link
Copy Markdown
Contributor

What

  • Enabled default error logs

Summary by CodeRabbit

  • Chores
    • Updated demo environment logging configuration to explicitly enable error-level logging for improved diagnostic visibility and clearer error tracking during development.

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

Signed-off-by: Krishna Waske <krishna.waske@ayanworks.com>
@GHkrishna GHkrishna self-assigned this Nov 24, 2025
@coderabbitai

coderabbitai Bot commented Nov 24, 2025

Copy link
Copy Markdown

Walkthrough

A simple update to the .env.demo file that replaces a commented-out Prisma logging configuration with an active setting. The PRISMA_LOGS environment variable is now explicitly set to error with accompanying documentation, replacing the previously commented example that listed multiple log types.

Changes

Cohort / File(s) Change Summary
Environment Configuration
\.env\.demo
Replaces commented Prisma logging config with active setting defaulting to error level

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

enhancement

Suggested reviewers

  • tipusinghaw

Poem

🐰 A humble log level, set with care,
From comments dark to values fair,
Error whispers now ring loud,
Configuration stands proud! 📝✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: default error logs enabled' directly reflects the main change: activating the PRISMA_LOGS configuration with error-level logging as the default.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/prisma-logging

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.

@GHkrishna GHkrishna added the chore For chores like linting, auto generated files or other minor tasks label Nov 24, 2025
Signed-off-by: Krishna Waske <krishna.waske@ayanworks.com>
@sonarqubecloud

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
.env.demo (1)

224-225: Align formatting with file convention: remove spaces around equals sign.

The line PRISMA_LOGS = error includes spaces around the =, which deviates from the convention used throughout the rest of the file (e.g., API_GATEWAY_PROTOCOL=http, SENDGRID_API_KEY=). This also triggered a static analysis warning from dotenv-linter.

Apply this diff to maintain consistency:

-# Prisma log type. Default set to error
-PRISMA_LOGS = error
+# Prisma log type. Default set to error
+PRISMA_LOGS=error
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 48a4342 and 42a2b6c.

📒 Files selected for processing (1)
  • .env.demo (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: KambleSahil3
Repo: credebl/platform PR: 1518
File: libs/prisma-service/prisma/data/credebl-master-table/credebl-master-table.json:11-20
Timestamp: 2025-11-17T08:03:16.180Z
Learning: In the credebl/platform repository, seed data files like libs/prisma-service/prisma/data/credebl-master-table/credebl-master-table.json intentionally contain empty strings for sensitive fields (email, username, passwords, API keys, etc.) because these values are populated from .env file at runtime, not hardcoded in the JSON.
🪛 dotenv-linter (4.0.0)
.env.demo

[warning] 225-225: [SpaceCharacter] The line has spaces around equal sign

(SpaceCharacter)

@GHkrishna
GHkrishna merged commit 3ab4ac6 into main Nov 24, 2025
8 checks passed
@GHkrishna
GHkrishna deleted the feat/prisma-logging branch November 24, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore For chores like linting, auto generated files or other minor tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants