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

Chenyueyan code patch 1 #11

Open

chenyueyan-code wants to merge 6 commits into main from chenyueyan-code-patch-1

Owner

chenyueyan-code commented Dec 24, 2020

No description provided.

chenyueyan-code added 6 commits

December 22, 2020 17:06


          Create selects.js

74d50fb


          Create test6.js

6da40d5


          Update test6.js

7c41686


          Update test6.js

e43fd41


          Update test6.js

f4cd811


          Update test6.js

e5f2d4f

ghost commented Dec 24, 2020

DeepCode's analysis on #e5f2d4 found:

⚠️ 4 warnings 👇

Top issues

Description	Example fixes
Two subsequent branches of an if statement have duplicate bodies (here and here). This may be caused by a copy-paste error. If this usage is intentional, consider merging the branches to avoid code duplication. Occurrences: test6.js:13	🔧 Example fixes
The variable obj (declared here) may be uninitialized when used here. Occurrences: test6.js:20	🔧 Example fixes
Assignment of a constant is probably not intended as a condition. Occurrences: test6.js:33	🔧 Example fixes

👉 View analysis in DeepCode’s Dashboard | Configure the bot

lgtm-com bot commented Dec 24, 2020

This pull request introduces 4 alerts when merging e5f2d4f into 29e4cff - view on LGTM.com

new alerts:

3 for Unused variable, import, function or class
1 for Missing rate limiting

sonarqubecloud bot commented Jun 27, 2022

SonarCloud Quality Gate failed.

6 Bugs
0 Vulnerabilities
1 Security Hotspot
23 Code Smells

No Coverage information
0.0% Duplication

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

Labels

None yet