Skip to content

Move vscode utils to /common/vscode#2388

Merged
charisk merged 6 commits intomainfrom
charisk/move-common-vscode-files
Apr 28, 2023
Merged

Move vscode utils to /common/vscode#2388
charisk merged 6 commits intomainfrom
charisk/move-common-vscode-files

Conversation

@charisk
Copy link
Copy Markdown
Contributor

@charisk charisk commented Apr 28, 2023

Moved VS Code specific utility files to /common/vscode

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.

@charisk charisk added the secexp label Apr 28, 2023
@charisk charisk requested review from a team as code owners April 28, 2023 09:36
@charisk charisk force-pushed the charisk/move-common-vscode-files branch from ec1c13b to b034c31 Compare April 28, 2023 10:15
norascheuch
norascheuch approved these changes Apr 28, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@norascheuch norascheuch left a comment

Choose a reason for hiding this comment

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

LGTM

@charisk charisk merged commit 27ebc3e into main Apr 28, 2023
@charisk charisk deleted the charisk/move-common-vscode-files branch April 28, 2023 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@norascheuch norascheuch norascheuch approved these changes

Assignees

No one assigned

Labels

secexp

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@charisk @norascheuch