Update README with VS Code Marketplace link and installation notes#24
Update README with VS Code Marketplace link and installation notes#24hoangsonww merged 2 commits intomasterfrom
Conversation
…e to README Co-authored-by: Kailie Le <davn0376@gmail.com>
…stops parsing 2>/dev/null as a path
There was a problem hiding this comment.
Code Review
This pull request updates the README.md for the VS Code extension by adding a marketplace link and a notice regarding the global installation of the Forge runtime. A review comment suggests adding an ## Installation header to ensure that internal documentation links within the extension function correctly by providing a valid anchor.
| > [!TIP] | ||
| > Marketplace Link: [https://marketplace.visualstudio.com/items?itemName=hoangsonw.forge-agentic-coding-cli](https://marketplace.visualstudio.com/items?itemName=hoangsonw.forge-agentic-coding-cli). | ||
|
|
||
| > [!IMPORTANT] |
There was a problem hiding this comment.
The extension's source code (specifically in extension.ts) contains a link to the documentation with an anchor #installation. However, the README currently lacks an Installation header, which means the link will not navigate to the correct section when clicked from the extension's warning message. Adding an ## Installation header here will fix this and provide a clear section for the installation notes.
| > [!IMPORTANT] | |
| ## Installation | |
| > [!IMPORTANT] |
This pull request makes improvements to the release workflow and documentation for the VS Code extension. The main changes ensure cross-platform compatibility in the workflow by specifying the shell, and enhance the README with installation guidance and a marketplace link.
Workflow improvements:
Install dependenciessteps in.github/workflows/release.ymlto explicitly useshell: bash, ensuring that on Windows runners, Git Bash is used instead of PowerShell. This avoids parsing errors with bash-specific commands and improves cross-platform reliability. [1] [2] [3]Documentation enhancements:
vscode-extension/README.md, instructing users to install the Forge runtime globally before using the extension, since it is not bundled.