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

Js/Py/Rb: Report any extracted file as successfully extracted #15256

Merged

Conversation

sidshank
Copy link
Contributor

@sidshank sidshank commented Jan 8, 2024

The files scanned section of the tool status page is currently pessimistic about which files are scanned. If there is a single extractor warning in the file, the file is marked as not scanned.

The changes in this pull request modify the CodeQL diagnostic queries to report all files inspected by the extractor, such that partially extracted files are accounted for in the diagnostics data presented on the Tool Status Page.

@sidshank sidshank marked this pull request as ready for review January 17, 2024 02:50
@sidshank sidshank requested review from a team as code owners January 17, 2024 02:50
@sidshank sidshank requested a review from henrymercer January 17, 2024 02:51
hvitved
hvitved previously approved these changes Jan 17, 2024
erik-krogh
erik-krogh previously approved these changes Jan 17, 2024
tausbn
tausbn previously approved these changes Jan 17, 2024
henrymercer
henrymercer previously approved these changes Jan 17, 2024
@sidshank
Copy link
Contributor Author

@hvitved @henrymercer @tausbn @erik-krogh I needed to add change notes - this is now done. My apologies for the review dismissals. Could you please take one more look and add your reviews? 🙏🏼

@sidshank sidshank merged commit 2d71294 into github:main Jan 17, 2024
27 checks passed
@sidshank sidshank deleted the change/adjust-extracted-files-diagnostics branch January 17, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants