Skip to content

Drop 'tech-inventory' and 'code duplication' queries from the standard query suites#4385

Merged
alexet merged 2 commits intogithub:mainfrom
aibaars:drop-queries
Oct 2, 2020
Merged

Drop 'tech-inventory' and 'code duplication' queries from the standard query suites#4385
alexet merged 2 commits intogithub:mainfrom
aibaars:drop-queries

Conversation

@aibaars
Copy link
Contributor

@aibaars aibaars commented Oct 1, 2020

hvitved
hvitved previously approved these changes Oct 2, 2020
Copy link
Contributor

@hvitved hvitved left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 for C#.

tausbn
tausbn previously approved these changes Oct 2, 2020
Copy link
Contributor

@tausbn tausbn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python 👍

erik-krogh
erik-krogh previously approved these changes Oct 2, 2020
Copy link
Contributor

@erik-krogh erik-krogh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JavaScript 👍

jbj
jbj previously approved these changes Oct 2, 2020
Copy link
Contributor

@jbj jbj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 and 🎉 from C/C++

@jbj
Copy link
Contributor

jbj commented Oct 2, 2020

The failing tests will hopefully get fixed by https://github.com/github/semmle-code/pull/37899 once that's merged.

@aibaars aibaars dismissed stale reviews from jbj, erik-krogh, tausbn, and hvitved via daa1bcc October 2, 2020 15:23
@alexet alexet merged commit 30ed6a0 into github:main Oct 2, 2020
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.

6 participants