Skip to content

Rename Query History Actions#12000

Merged
norascheuch merged 1 commit intomainfrom
nora/update-query-history-actions
Feb 10, 2023
Merged

Rename Query History Actions#12000
norascheuch merged 1 commit intomainfrom
nora/update-query-history-actions

Conversation

@norascheuch
Copy link
Copy Markdown
Contributor

The next version of the VS Code extension will rename some actions on the query history menu.

@norascheuch norascheuch requested a review from a team as a code owner January 26, 2023 16:41
@norascheuch norascheuch added the ready-for-doc-review This PR requires and is ready for review from the GitHub docs team. label Jan 27, 2023
Copy link
Copy Markdown
Member

@koesie10 koesie10 left a comment

Choose a reason for hiding this comment

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

LGTM, but we'll probably need to wait until the extension has been released before merging this.

Copy link
Copy Markdown
Contributor

@felicitymay felicitymay left a comment

Choose a reason for hiding this comment

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

I'm not sure of the release schedule for the VS Code extension but agree that this should be merged to ensure that the changes are published at the right moment.

We update the docs for every CodeQL CLI release, publishing from the relevant release candidate branch.

Copy link
Copy Markdown
Contributor

@felicitymay felicitymay left a comment

Choose a reason for hiding this comment

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

Thanks for your patience with my misunderstanding @norascheuch.

This looks ready to merge when you have a date for the release.

@charisk
Copy link
Copy Markdown
Contributor

charisk commented Feb 8, 2023

A new release was published on February the 3rd (https://github.com/github/vscode-codeql/releases/tag/v1.7.8), shall we merge this?

@felicitymay
Copy link
Copy Markdown
Contributor

A new release was published on February the 3rd (https://github.com/github/vscode-codeql/releases/tag/v1.7.8), shall we merge this?

If you merge this PR the updated articles will be published with the next release of the CodeQL CLI in about two weeks. Is that okay?

@charisk
Copy link
Copy Markdown
Contributor

charisk commented Feb 8, 2023

A new release was published on February the 3rd (https://github.com/github/vscode-codeql/releases/tag/v1.7.8), shall we merge this?

If you merge this PR the updated articles will be published with the next release of the CodeQL CLI in about two weeks. Is that okay?

That sounds good to me. @norascheuch, @koesie10?

@norascheuch norascheuch merged commit d55e9d5 into main Feb 10, 2023
@norascheuch norascheuch deleted the nora/update-query-history-actions branch February 10, 2023 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation ready-for-doc-review This PR requires and is ready for review from the GitHub docs team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants