Skip to content

docs: fix #vscode link in ide integration page#8833

Merged
jdx merged 1 commit intojdx:mainfrom
jedymatt:patch-1
Mar 31, 2026
Merged

docs: fix #vscode link in ide integration page#8833
jdx merged 1 commit intojdx:mainfrom
jedymatt:patch-1

Conversation

@jedymatt
Copy link
Copy Markdown
Contributor

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 31, 2026

Greptile Summary

This PR fixes a broken anchor reference in docs/ide-integration.md. The text #vscode was a bare hash-prefixed string that rendered as literal text rather than a hyperlink. The fix wraps it in proper Markdown link syntax ([VSCode](#vscode)), correctly pointing to the ## VSCode section that exists lower in the same page.

Confidence Score: 5/5

Safe to merge — documentation-only fix with no logic changes.

The change is a minimal, correct documentation fix. The anchor target #vscode matches the existing ## VSCode section heading. No code is modified and no regressions are possible.

No files require special attention.

Important Files Changed

Filename Overview
docs/ide-integration.md One-line fix converting a bare #vscode text reference into a proper Markdown link [VSCode](#vscode) that correctly targets the ## VSCode section on the same page.

Reviews (1): Last reviewed commit: "Fix #vscode link in docs" | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation in docs/ide-integration.md to use a proper markdown link for the VSCode section. The review feedback suggests a more concise phrasing for the link and the addition of a comma to improve sentence flow and readability.

:::

On Linux this is read when logging into the machine, so changing it requires logging out and back in for it to work. See #vscode below
On Linux this is read when logging into the machine, so changing it requires logging out and back in for it to work. See [VSCode](#vscode) below
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.

medium

The phrasing here is slightly repetitive ("See VSCode below for how to get VSCode..."). Using "below" as the link text is more concise and consistent with other internal links in this document (e.g., line 12). Additionally, adding a comma after "On Linux" improves readability.

Suggested change
On Linux this is read when logging into the machine, so changing it requires logging out and back in for it to work. See [VSCode](#vscode) below
On Linux, this is read when logging into the machine, so changing it requires logging out and back in for it to work. See [below](#vscode)

@jdx jdx merged commit 3774ac3 into jdx:main Mar 31, 2026
36 checks passed
mise-en-dev added a commit that referenced this pull request Apr 1, 2026
### 🚀 Features

- add azd (Azure Developer CLI) to registry by @rajeshkamal5050 in
[#8828](#8828)

### 🐛 Bug Fixes

- **(aqua)** skip registry lookup for linked versions in list_bin_paths
by @nikobockerman in [#8801](#8801)
- **(rust)** handle rustup check exit code 100 as non-error by @shalk in
[#8832](#8832)
- **(task)** resolve bare aliases in monorepo with config_roots by
@nkakouros in [#8819](#8819)
- show usage help when long_about is defined w/o args/flags by
@nkakouros in [#8824](#8824)

### 📚 Documentation

- fix serif font in sidebar and increase heading sizes by @jdx in
[#8831](#8831)
- fix #vscode link in ide integration page by @jedymatt in
[#8833](#8833)
- fix nested Markdown code fences by @muzimuzhi in
[#8835](#8835)

### New Contributors

- @shalk made their first contribution in
[#8832](#8832)
- @jedymatt made their first contribution in
[#8833](#8833)
- @nikobockerman made their first contribution in
[#8801](#8801)
- @rajeshkamal5050 made their first contribution in
[#8828](#8828)

## 📦 Aqua Registry Updates

#### New Packages (2)

- [`gastownhall/beads`](https://github.com/gastownhall/beads)
- [`getdbt.com/dbt-fusion`](https://github.com/getdbt.com/dbt-fusion)

#### Updated Packages (2)

- [`Azure/azure-dev`](https://github.com/Azure/azure-dev)
- [`magefile/mage`](https://github.com/magefile/mage)
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.

2 participants