Skip to content

Refactor logging to use console.debug for clarity#4

Merged
roble merged 1 commit into
mainfrom
dev
Apr 10, 2026
Merged

Refactor logging to use console.debug for clarity#4
roble merged 1 commit into
mainfrom
dev

Conversation

@roble

@roble roble commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

This pull request makes a minor change to the logging in resources/js/app.ts by switching from console.log to console.debug for two initialization messages. This helps reduce log noise in production environments.

  • Logging changes:
    • Changed console.log to console.debug in the setup function to make the "Roadmap module loaded" message less prominent.
    • Changed console.log to console.debug in the afterMount function to make the "Roadmap module after mount logic executed" message less prominent.

Copilot AI review requested due to automatic review settings April 10, 2026 19:58

Copilot AI 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.

Pull request overview

This PR refactors two initialization log statements in the Roadmap module to use console.debug instead of console.log, aiming to make these messages less prominent.

Changes:

  • Updated setup() initialization message from console.log to console.debug
  • Updated afterMount() initialization message from console.log to console.debug

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread resources/js/app.ts
Comment thread resources/js/app.ts
@roble roble merged commit 902a811 into main Apr 10, 2026
6 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.

2 participants