-
Notifications
You must be signed in to change notification settings - Fork 1.7k
LGTM deprecation: Update basic queries to use VS Code #11468
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
LGTM deprecation: Update basic queries to use VS Code #11468
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C++ generally looks good to me. One question.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Java LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, one minor thing.
docs/codeql/codeql-language-guides/basic-query-for-java-code.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Tony Torralba <atorralba@users.noreply.github.com>
Ruby 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JS 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally quite nice, only one typo.
I wonder if we should find a smaller DB than saltstack/salt
, but on the other hand it is nice to see a bunch of results and then a small bunch of results after refining the query..
docs/codeql/reusables/vs-code-basic-instructions/run-quick-query-1.rst
Outdated
Show resolved
Hide resolved
docs/codeql/reusables/vs-code-basic-instructions/run-quick-query-1.rst
Outdated
Show resolved
Hide resolved
docs/codeql/reusables/vs-code-basic-instructions/run-quick-query-2.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Michael B. Gale <mbg@github.com>
docs/codeql/reusables/vs-code-basic-instructions/run-quick-query-2.rst
Outdated
Show resolved
Hide resolved
👋🏻 I believe that I've applied all changes. I'm planning to merge this PR before I stop work at the end of my (UK) day. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
I checked a few times, but you must have done it later. What I would normally expect is that you re-request a review from me (in the top right reviewers panel) and then I get notified and can approve your changes. Then you do not have to merge in the "changes requested" state :-) |
Hi @yoff. Thanks for letting me know the expected etiquette for this situation, it's much appreciated. We rarely use that feature in the repositories I usually work in and I overlooked it. If there are changes that I missed, or you have feedback on the changes I made, please do go ahead and add them here. I'd be happy to open a new PR with any corrections. |
I think all my comments are addressed appropriately 👍 |
This pull request updates the basic query introduction articles to use VS Code in pace of LGTM.com's query console. This is a replacement for: #11423.
Please tick the list below to confirm that your team is happy with the changes.
The general approach was reviewed and agreed in: #11423.
For an HTML preview of the resulting docs, look at the Docs - Generate Sphinx check. You should find an artefact that you can download, so you can view the HTML generated from the
rst
files.