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

Console.log fix #13

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

philipjonsen
Copy link

Just delete deepsource files

deepsource-io bot and others added 4 commits September 22, 2023 16:37
It is considered a best practice to avoid the use of any `console` methods in JavaScript code that will run on the browser.

**NOTE:** If your repository contains a server side project, you can add `"nodejs"` to the `environment` property of analyzer meta in `.deepsource.toml`.
This will prevent this issue from getting raised.
Documentation for the analyzer meta can be found [here](https://docs.deepsource.com/docs/analyzers-javascript#meta).
Alternatively, you can silence this issue for your repository [as shown here](https://deepsource.com/blog/releases-issue-actions).

If a specific `console` call is meant to stay for other reasons, you can add [a skipcq comment](https://docs.deepsource.com/docs/issues-ignore-rules#silencing-a-specific-issue) to that line.
This will inform other developers about the reason behind the log's presence, and prevent DeepSource from flagging it.
refactor: remove the console expressions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant