Skip to content

fix(sidebars): remove accidental console.log - #330

Open
mdryaaan wants to merge 1 commit into
koordinator-sh:mainfrom
mdryaaan:fix/sidebars-console-log
Open

mdryaaan wants to merge 1 commit into
koordinator-sh:mainfrom
mdryaaan:fix/sidebars-console-log

Conversation

@mdryaaan

Copy link
Copy Markdown

Problem

sidebars.js line 179 has console.log(JSON.stringify(sidebars, null, 2)). Docusaurus requires this file as a Node module, so this executes on every npm run build, npm run start, and docusaurus docs:version call. It prints ~150 lines of JSON to stdout, flooding build output and CI logs.

Expected

The console.log should not exist in a config file.

  • Remove console.log(JSON.stringify(sidebars, null, 2)) from the end of sidebars.js — this line executed on every build and dev-server start, printing ~150 lines of JSON to stdout and flooding CI logs

Files Changed

  • sidebars.js — deleted line 179 (the console.log call)

@koordinator-bot
koordinator-bot Bot requested review from ZiMengSheng and hormes May 13, 2026 03:54
@koordinator-bot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign zimengsheng after the PR has been reviewed.
You can assign the PR to them by writing /assign @zimengsheng in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: mdryaan <alikhurshid842001@gmail.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant