Skip to content

feat: add greeting command and startup log to vscode-extras - #327918

Open
denizguney wants to merge 2 commits into
microsoft:mainfrom
denizguney:main
Open

feat: add greeting command and startup log to vscode-extras#327918
denizguney wants to merge 2 commits into
microsoft:mainfrom
denizguney:main

Conversation

@denizguney

Copy link
Copy Markdown

The sayHello command—executable from the command palette—and a startup log channel message have been added to the extension.

The `sayHello` command—executable from the command palette—and a startup log channel message have been added to the extension.
Copilot AI review requested due to automatic review settings July 28, 2026 22:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a greeting command and activation log to the VS Code Extras development extension.

Changes:

  • Logs successful extension activation.
  • Registers a greeting command that displays a notification.

private _npmFeature: NpmUpToDateFeature | undefined;

constructor(_context: vscode.ExtensionContext) {
constructor(context: vscode.ExtensionContext) {


disposables.push(
vscode.commands.registerCommand('vscode-extras.sayHello', () => {

disposables.push(
vscode.commands.registerCommand('vscode-extras.sayHello', () => {
vscode.window.showInformationMessage('Hey friend! VS Code Extras is online..');
Comment on lines +20 to +23

this._output.info('The VS Code Extras extension has been successfully started and is active!');


Refactor constructor parameter and improve log message.

@denizguney denizguney left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks for the feedback! I've updated the code and addressed the formatting and parameter requirements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants