Skip to content

Queries Panel: run local query from file tab#2544

Merged
norascheuch merged 1 commit intomainfrom
nora/queries-panel-file-tab
Jun 26, 2023
Merged

Queries Panel: run local query from file tab#2544
norascheuch merged 1 commit intomainfrom
nora/queries-panel-file-tab

Conversation

@norascheuch
Copy link
Copy Markdown
Contributor

Adds a run icon to .ql-files and runs local query.

Checklist

  • 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.

@norascheuch norascheuch requested a review from a team as a code owner June 23, 2023 14:33
@norascheuch norascheuch force-pushed the nora/queries-panel-file-tab branch from a913ed0 to d4e9d64 Compare June 23, 2023 14:48
Copy link
Copy Markdown
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.

Cool! I have a question about the visibility of this command, but looks good otherwise.

(I'm never hugely confident in my reviews though, so I wouldn't mind a second opinion for reviewing/approving 😺)

Copy link
Copy Markdown
Contributor

@robertbrignull robertbrignull left a comment

Choose a reason for hiding this comment

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

Code changes LGTM

I have the same question as @shati-patel about whether we want to feature flag this or not. I agree it pretty independent and could be nice to release it now.

@norascheuch norascheuch force-pushed the nora/queries-panel-file-tab branch from d4e9d64 to 10d8bbf Compare June 26, 2023 12:48
@norascheuch norascheuch enabled auto-merge June 26, 2023 13:54
@norascheuch norascheuch merged commit 75d2f76 into main Jun 26, 2023
@norascheuch norascheuch deleted the nora/queries-panel-file-tab branch June 26, 2023 14:09
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.

3 participants