Skip to content

Queries panel: Run all queries in folder (context menu)#2554

Merged
shati-patel merged 1 commit intomainfrom
shati-patel/run-all-context-menu
Jun 27, 2023
Merged

Queries panel: Run all queries in folder (context menu)#2554
shati-patel merged 1 commit intomainfrom
shati-patel/run-all-context-menu

Conversation

@shati-patel
Copy link
Copy Markdown
Contributor

@shati-patel shati-patel commented Jun 27, 2023

Adds a command to run all queries in a certain folder. This uses the existing runQueries command, which lets you run multiple queries against the selected local database.

image

⚠️ We don't have a corresponding command for running multiple variant analysis queries, so I haven't implemented that. It's probably a separate product discussion of whether we want to allow users to start multiple variant analysis runs in one click? 🤔

Checklist

N/A (feature-flagged for internal use)

  • 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
Queries panel: Run all queries in folder (context menu)
02b356c 
Adds a command to run all queries in a certain folder.
This uses the existing `runQueries` command, which lets you run multiple queries against the selected local database.
We don't have a corresponding command for running multiple variant analysis queries, so I haven't implemented that.
@shati-patel shati-patel requested a review from a team as a code owner June 27, 2023 14:08
Base automatically changed from nora/run-all-queries-panel to main June 27, 2023 14:37
robertbrignull
robertbrignull approved these changes Jun 27, 2023
View reviewed changes
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.

LGTM

🚢

@shati-patel shati-patel merged commit 9314b3b into main Jun 27, 2023
@shati-patel shati-patel deleted the shati-patel/run-all-context-menu branch June 27, 2023 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@robertbrignull robertbrignull robertbrignull approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@shati-patel @robertbrignull