Skip to content

Add .NET doc site link on the drasi-lib Other Languages page - #273

Open
danielgerlag wants to merge 1 commit into
mainfrom
danielgerlag-add-dotnet-docs-link
Open

Add .NET doc site link on the drasi-lib Other Languages page#273
danielgerlag wants to merge 1 commit into
mainfrom
danielgerlag-add-dotnet-docs-link

Conversation

@danielgerlag

Copy link
Copy Markdown
Contributor

Follow-up to #270. Adds a third language card for the .NET docs site next to the existing Node.js and Python cards on the drasi-lib landing page and the Other Languages left-nav page.

.NET docs: https://drasi-project.github.io/drasi-dotnet/

The Other Languages card grids now use card-grid--3 so the three cards sit in one row. Homepage, top nav, and Concepts are unchanged.

Follow-up to #270. Add a third language card for the .NET docs
site (https://drasi-project.github.io/drasi-dotnet/) next to the
existing Node.js and Python cards, and switch those grids to
three columns.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 9, 2026 22:16
@danielgerlag
danielgerlag marked this pull request as ready for review September 9, 2026 22:27
@danielgerlag
danielgerlag requested a review from a team September 9, 2026 22:27
@danielgerlag danielgerlag self-assigned this Sep 9, 2026

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.

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Pull request overview

Adds a .NET documentation card/link to the “Other Languages” sections so Node.js, Python, and .NET appear together in a 3-column card grid.

Changes:

  • Update “Other Languages” copy to include .NET.
  • Switch the card grid layout from 2 to 3 columns.
  • Add a new card linking to the .NET docs site.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/content/drasi-lib/other-languages/_index.md Adds .NET card/link and updates copy + grid layout on the Other Languages page.
docs/content/drasi-lib/_index.md Mirrors the .NET card/link and copy + grid layout on the drasi-lib landing page section.
Suppressed comments (1)

docs/content/drasi-lib/_index.md:1

  • The card grid markup is now duplicated in both the landing page and the Other Languages page (including the new .NET card). This increases the risk of the two sections drifting over time (copy, URLs, CSS classes, etc.). Consider extracting the card grid into a Hugo shortcode/partial and including it from both pages so updates happen in one place.
---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

</div>
</a>
<a href="https://drasi-project.github.io/drasi-dotnet/" target="_blank" rel="noopener noreferrer">
<div class="unified-card unified-card--reference">
Comment on lines +136 to 145
<a href="https://drasi-project.github.io/drasi-dotnet/" target="_blank" rel="noopener noreferrer">
<div class="unified-card unified-card--reference">
<div class="unified-card-icon"><i class="fab fa-microsoft"></i></div>
<div class="unified-card-content">
<h3 class="unified-card-title">.NET (<code>Drasi</code>)</h3>
<p class="unified-card-summary">Embed the Drasi continuous-query engine in a .NET application</p>
</div>
</div>
</a>
</div>

@ruokun-niu ruokun-niu 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.

nit: maybe we should highlight that users need to clone the repo and cd into the folder first, then choose either the shell script or the cargo build and dotnet run commands.
Image

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.

3 participants