Skip to content

Conversation

jf205
Copy link
Contributor

@jf205 jf205 commented Apr 24, 2020

Fixes https://github.com/github/semmle-docs/issues/48.

Adds a bit of information about path queries (in a reusable note (source)) to the data flow tutorials for the languages where it wasn't already mentioned.

In each case I added the note just below the section about "Global data flow". For example, see Analyzing data flow in C and C++.

(Note the CS changed slightly since it was approved--some work was shifted into https://github.com/github/semmle-docs/issues/68#issuecomment-618432397.)

@jf205 jf205 requested a review from shati-patel as a code owner April 24, 2020 06:53
@jf205 jf205 removed the request for review from shati-patel April 24, 2020 06:53
@jf205 jf205 changed the title docs: add note about path queries to data flow tutorials Learn CodeQL docs: add note about path queries to data flow tutorials Apr 24, 2020
Copy link
Contributor

@shati-patel shati-patel left a comment

Choose a reason for hiding this comment

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

The note might be more visible if you put it at the top of the file, but this is good too! 👍🏼

@jf205 jf205 merged commit 73c29ec into github:rc/1.24 Apr 27, 2020
@jf205 jf205 deleted the sd-48 branch April 27, 2020 08:13
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.

2 participants