-
-
Notifications
You must be signed in to change notification settings - Fork 793
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
Feasibility and DR: use of CodeQL for VS Code #5180
Comments
Hi @kiran98118, thank you for taking up this issue! Hfla appreciates you :) Do let fellow developers know about your:- You're awesome! P.S. - You may not take up another issue until this issue gets merged (or closed). Thanks again :) |
i. Availability: I am available on Sunday, Tuesday, Wednesday, Friday and Saturday (10 am to 5pm) |
@kiran98118 It looks like you forgot to move this to the in progress column on the board. I will move it for you. |
Thank you Bonnie.
…On Tue, Feb 13, 2024 at 4:07 PM Bonnie Wolfe ***@***.***> wrote:
@kiran98118 <https://github.com/kiran98118> It looks like you forgot to
move this to the in progress column on the board. I will move it for you.
—
Reply to this email directly, view it on GitHub
<#5180 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKIK4WG3SVHWQHU6QHTNHOLYTPIZLAVCNFSM6AAAAAA3LEZZ52VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSGUZTMNJWGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hi @kiran98118! It looks like you have moved this because you are finished with the issue and it is ready for review. However, I don't see a draft DR in a comment on this issue or a link to a DR on the How to Contribute to the Wiki page. If I've missed it, please let me know. Otherwise, please add the DR according to the instructions on the How to Contribute to the Wiki page. Feel free to ping me if you have any questions. I am moving this back to the Thanks for taking the time to contribute! |
Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.
If you need help, be sure to either: 1) place your issue in the You are receiving this comment because your last comment was before Monday, February 19, 2024 at 11:06 PM PST. |
Draft DR: Recommendation to adopt CodeQL for VS Code extension
|
|
CodeQL only supports C/C++, C#, Go, Java/Kotlin, JavaScript/TypeScript, Python, Ruby, Swift languages. If the code you're attempting to scan is not written in one of the supported languages, the CodeQL scan will fail automatically. This behavior is also observed when using the CodeQL extension within Visual Studio (VS). The tool does not have the capability to analyze or process code written in languages that are not explicitly supported by the CodeQL platform. |
Thank you @kiran98118 for your analysis and recommendation. |
Overview
We should consider whether to adopt the policy that developers should install the "CodeQL for VS Code" extension. Currently the default branch of the repository is scanned weekly and the changed files in each Pull Request are scanned, however there are advantages to recieving alerts prior to initiating the Pull Request process.
Action Items
ready for dev lead
label.Resources/Instructions
The text was updated successfully, but these errors were encountered: