Skip to content

Conversation

felicitymay
Copy link
Contributor

This PR adds query ids where they were missing and makes a few small text changes.

@Semmle/js - is there anything missing from this?

@felicitymay felicitymay added JS documentation Priority PR that should be reviewed and merged as a matter of priority. labels Aug 24, 2019
@felicitymay felicitymay added this to the 1.22 milestone Aug 24, 2019
@asger-semmle
Copy link
Contributor

Hang on, there's one more change note I'd like to get in. I'll open a PR against this PR.

@felicitymay
Copy link
Contributor Author

Thanks @asger-semmle

Copy link
Contributor

@asger-semmle asger-semmle left a comment

Choose a reason for hiding this comment

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

LGTM

@semmle-qlci semmle-qlci merged commit e37751e into github:master Aug 24, 2019
@felicitymay felicitymay deleted the 1.22/js-finalize-change-notes branch September 23, 2019 16:32
@felicitymay felicitymay restored the 1.22/js-finalize-change-notes branch September 23, 2019 16:32
@felicitymay felicitymay deleted the 1.22/js-finalize-change-notes branch September 23, 2019 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation JS Priority PR that should be reviewed and merged as a matter of priority.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants