-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Minor updates to CodeQL for VS Code docs for expansion of support for model editing #15748
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
Conversation
This was accidentally reintroduced during merge conflict resolution.
shati-patel
left a comment
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.
Updates look good! 📝
(Minor side note—not from this PR, just looking at the preview: There's a .. tip:: box on this page which isn't really styled correctly. Might be worth changing that to a .. pull-quote:: to match the other boxes?)
subatoi
left a comment
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.
Thank you, particularly for updating the screenshots ❤️
efff014
|
Thanks for the rapid reviews 💖 I've made the change @shati-patel requested, so this is probably ready to merge when one of you has time to give it a 👍🏻 |
subatoi
left a comment
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.
No reason to think CI won't pass 👍
There are only minor changes needed to the docs for the VS Code extension.
Reviewers, it would be great to get this merged before the next release candidate branch is created.