-
Notifications
You must be signed in to change notification settings - Fork 8
Created a NodeJS Console app demo without Copilot Chat #8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
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
There was a problem hiding this 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
Co-authored-by: Jürgen Efeish <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this 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` | |
There was a problem hiding this comment.
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.'
| [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.
This is for a customer who cannot enable Copilot Chat
This pull request primarily focuses on updates to the
README.md
files in thedemos
directory, specifically for theCommand-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 theCommand-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:demos/Command-Execution-WebApp-NodeJS-CopilotChat/README.md
: The file was renamed fromdemos/Command-Execution-WebApp-NodeJS/README.md
and updated to specify that it is the Copilot Chat version of the application. This makes it easier for users to differentiate between the two versions.demos/Command-Execution-WebApp-NodeJS-CopilotChat/README.md
: The instructions for building and running the application were revised for clarity, including changing "Step-by-step guides" to "Step-by-step guide" and ensuring all prerequisites are met.