-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
[BUG] Fix LGTM/CodeQL alerts #960
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
Comments
is there any one working on it |
Nope, not currently. Do you wanna work on this? 🙂 |
sure I will try My best sir 😊 |
Cool, thank you! Let us know if you need any help. |
sir, i can't find the repository can you give me the name of it |
Here you can see all the alerts/warnings: https://lgtm.com/projects/g/TheAlgorithms/C/context:cpp |
"I’d like to work on fixing LGTM alerts in [file.c]. Please assign me!" |
Description
There are 10+ CodeQL alerts on the whole repository. This should be fixed immediately.
Expected behavior
CodeQL/LGTM alerts should be fixed.
Actual behavior
We have 10+ CodeQL/LGTM alerts.
Possible fix
No response
Steps to reproduce
Not applicable here, IMO.
Context
This makes our code less elegant and professional. We should try to use the best code possible.
Additional information
Anyone who wants to work on this, feel free to do so. Just make sure you comment here before working on the issue. 🙂
The text was updated successfully, but these errors were encountered: