search

X-lab2017 / open-leaderboard

OpenLeaderboard
https://open-leaderboard.x-lab.info
20 stars 23 forks source link

[Chore] to save the messy console #64

Closed tyn1998 closed 1 year ago

tyn1998 commented 1 year ago

I think console output for test purpose should be removed from the codebase:

image

😆

andyhuang18 commented 1 year ago

Yes~It is time to remove the testing output

wj23027 commented 1 year ago

I agree with you~

frank-zsy commented 1 year ago

I think we can put a variable in the code to enable or disable the output. console output is quite useful for debugging, we can set it to disable on release automatically.

kunal8411 commented 1 year ago

Hi Team,

I have raised one PR for this issue, can you please check.

PR: https://github.com/X-lab2017/open-leaderboard/pull/93 i have created one env file and stored one variable in that file as "NODE_ENV" of the value of NODE_ENV is development then only we will be able to see the consoles for debugging, other than dev env the consoles will be hidden.

andyhuang18 commented 1 year ago

Thank you for your contribution, I will review your commit as soon as possible!