Skip to content

(Minor) Remove outdated comment#1453

Merged
shati-patel merged 1 commit intomainfrom
shati-patel/delete-comment
Aug 4, 2022
Merged

(Minor) Remove outdated comment#1453
shati-patel merged 1 commit intomainfrom
shati-patel/delete-comment

Conversation

@shati-patel
Copy link
Copy Markdown
Contributor

Very tiny fix, just spotted in passing! 🙈

I've deleted the comment In the future, we may support cancelling remote queries, but this is not a short term plan., since we do in fact support cancelling remote queries 😄 (as implemented here)

Checklist

n/a

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@shati-patel shati-patel requested a review from a team as a code owner August 4, 2022 13:11
Copy link
Copy Markdown
Contributor

@angelapwen angelapwen left a comment

Choose a reason for hiding this comment

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

Fastest review of my life 🙆‍♀️

@shati-patel shati-patel enabled auto-merge (squash) August 4, 2022 13:23
@shati-patel shati-patel merged commit 4cbf104 into main Aug 4, 2022
@shati-patel shati-patel deleted the shati-patel/delete-comment branch August 4, 2022 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants