Skip to content

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

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

fix(sidebars): remove accidental console.log#330
mdryaan wants to merge 1 commit into
koordinator-sh:mainfrom
mdryaan:fix/sidebars-console-log

Conversation

@mdryaan
Copy link
Copy Markdown

@mdryaan mdryaan commented May 13, 2026

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>
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