Skip to content

Conversation

bryantson
Copy link
Collaborator

This is for a customer who cannot enable Copilot Chat

This pull request primarily focuses on updates to the README.md files in the demos directory, specifically for the Command-Execution-WebApp-NodeJS demo. The changes provide clearer instructions for building and running the NodeJS application that executes shell commands in a browser, and also include the creation of a separate version of the application that incorporates Copilot Chat.

Here are the key changes:

Updates to README.md:

  • README.md: The description of the Command-Execution-WebApp-NodeJS demo has been updated to include versions with and without Copilot Chat. This provides users with more options depending on their preferences or constraints.

Changes to Command-Execution-WebApp-NodeJS demo:

Copy link

Choose a reason for hiding this comment

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

Note: Remove the extra ` from the README.md

You can follow the instruction to create a new `README.md`` page.
Step 8: Updating history view to make them executable upon click

minor spelling/grammar tweaks
CallMeGreg
CallMeGreg previously approved these changes Jan 17, 2024
Copy link
Contributor

@CallMeGreg CallMeGreg left a comment

Choose a reason for hiding this comment

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

This is awesome @bryantson ! ✨

I dropped in some very minor spelling fixes. This will be a great exercise with a bit more structure than https://github.com/ps-copilot-sandbox/copilot-demo-website

Copy link
Collaborator

@snsinahub snsinahub left a comment

Choose a reason for hiding this comment

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

LGTM

arilivigni
arilivigni previously approved these changes Jan 19, 2024
snsinahub
snsinahub previously approved these changes Jan 22, 2024
@bryantson bryantson dismissed stale reviews from snsinahub and arilivigni via c6cce53 May 8, 2024 20:48
@bryantson bryantson requested a review from a team as a code owner May 8, 2024 20:48
@arilivigni arilivigni requested a review from Copilot November 8, 2024 14:19
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 49 out of 49 changed files in this pull request and generated 1 suggestion.

Comments skipped due to low confidence (1)

demos/Command-Execution-WebApp-NodeJS-CopilotChat/README.md:3

  • The phrase 'Interative Development Editor' should be corrected to 'Integrated Development Environment'.
This is a simple NodeJS application that executes shell commands in a browser using **Github Copilot**. Please follow instructions below to build and run the application. This demo will primarily drive using **Github Copilot Chat** to build the application. It is possible to use other Interative Development Editor (IDEs) that support GitHub Copilot and GitHub Copilot Chat, but it is easier to follow and better experience to use VS Code.

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

| [Simple Javascript Calculator](demos/Javascript-calculator) | A simple calculator that adds two numbers together |
| [Node Calculator](demos/Node-calculator) | A simple calculator that adds two numbers together |
| [NodeJS app that runs shell commands with Copilot Chat](demos/Command-Execution-WebApp-NodeJS) | A NodeJS webapp that runs shell commands with Copilot Chat |
| [NodeJS app that runs shell commands without Copilot Chat](demos/Command-Execution-WebApp-NodeJS) | A NodeJS webapp that runs shell commands without Copilot Chat. Use this for customer `who does not want to use Copilot Chat or cannot` |
Copy link
Preview

Copilot AI Nov 8, 2024

Choose a reason for hiding this comment

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

The correct phrasing should be 'Use this for customers who do not want to use Copilot Chat or cannot.'

Suggested change
| [NodeJS app that runs shell commands without Copilot Chat](demos/Command-Execution-WebApp-NodeJS) | A NodeJS webapp that runs shell commands without Copilot Chat. Use this for customer `who does not want to use Copilot Chat or cannot` |
| [NodeJS app that runs shell commands without Copilot Chat](demos/Command-Execution-WebApp-NodeJS) | A NodeJS webapp that runs shell commands without Copilot Chat. Use this for customers who do not want to use Copilot Chat or cannot |

Copilot uses AI. Check for mistakes.

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.

5 participants