Skip to content
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.

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

Remove console.logs and improve try/catch through the whole repo #2500

Open
alonakos opened this issue Aug 19, 2024 · 1 comment
Open

Remove console.logs and improve try/catch through the whole repo #2500

alonakos opened this issue Aug 19, 2024 · 1 comment
Labels
linting & formatting Issues related to eslint and prettier config Priority: Low

Comments

@alonakos
Copy link

Describe the bug
Through the code base, there are numerous console.log statements within the try/catch block. See the example here.

We would like to handle error catch more gracefully than the console. logging the error statement, and avoid using console.log in production code. We can also remove the try/catch where they are redundant and keep them where useful.

Expected behavior
Clean the code to handle errors gracefully.

@robyngit
Copy link
Member

Extra context: avoiding console.log in our production code is part transitioning our codebase to use the airbnb style guide, see CONTRIBUTING - linting.

@robyngit robyngit added linting & formatting Issues related to eslint and prettier config Priority: Low and removed bug labels Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linting & formatting Issues related to eslint and prettier config Priority: Low
Projects
None yet
Development

No branches or pull requests

2 participants