New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix: removed extra prints in unit test #1152

Merged

lucifercr07 merged 1 commit into DiceDB:master from apoorvyadav1111:fix/remove-print-unit-test

Oct 20, 2024

Contributor

apoorvyadav1111 commented Oct 19, 2024

Hi team,

These print statement floods the log when unit tests fail and thus make the logs difficult to understand.

Thanks
Apoorv


          removed extra prints in unit test

de5d4b3

apoorvyadav1111 added the bug label

Contributor Author

apoorvyadav1111 commented Oct 20, 2024

Hi @lucifercr07, Can you please review this?

lucifercr07 approved these changes

View reviewed changes

lucifercr07 merged commit 1ca7492 into DiceDB:master

2 checks passed

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

Labels

bug