-
Notifications
You must be signed in to change notification settings - Fork 16
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
Java: Add experimental queries #80
Conversation
237274b
to
9efb681
Compare
9efb681
to
2856834
Compare
Not sure, seems like these are combining local and remote sources. Can they be removed @GeekMasher ?
You mean publishing them into a
I think the field/PS teams used them for PoCs. @GeekMasher do you think using Local Threat Models can get you what you were using these queries for?
Agreed
I wish my commits were as clean and organized as yours, it was a breeze to review! |
@pwntester : Thank you very much for the review and the kinds comments :-)
|
In this PR all the java experimental queries from the codeql repo is added to the community pack.
Some things to consider when review'ing.
TestUtilities
needed to be included as they currently live in the codeqltest
qlpack (which is not published). Maybe we should consider to move these into a pack where they can be referenced from this repo.Reviewing on a commit by commit basis is encouraged.