Skip to content

Conversation

@ShreyPurohit
Copy link
Contributor

@ShreyPurohit ShreyPurohit commented Mar 19, 2025

Related Issue: #146

VSCode Extension: Gitingest Extension

Underlying GitHub Repo: gitingest-vsextension

Changes:

  • Update README.md
  • Add link to VSCode Extension in src/server/templates/components/navbar.jinja

image


Basic Usage

Context-Menu-Usage


Setup Guide

Setup-Guide

Copy link
Contributor

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.

Pull Request Overview

This PR adds a VS Code extension link and icon to the site navbar and updates the README to advertise the new VS Code extension.

  • Added a VS Code Extension button (with inline SVG) to the navbar component
  • Extended README link list and features section to include the VS Code extension

Reviewed Changes

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

File Description
src/server/templates/components/navbar.jinja Inserted a new VS Code Extension link with inline SVG icon
README.md Appended VS Code Extension to links and feature list
Comments suppressed due to low confidence (2)

README.md:16

  • [nitpick] Use consistent product naming—change VS-Code Extension to VS Code Extension to match other references.
[gitingest.com](https://gitingest.com) · [Chrome Extension](https://chromewebstore.google.com/detail/adfjahbijlkjfoicpjkhjicpjpjfaood) · [Firefox Add-on](https://addons.mozilla.org/firefox/addon/gitingest) · [VS-Code Extension](https://marketplace.visualstudio.com/items?itemName=iamshreydxv.gitingest)

README.md:29

  • The nested bullets under "VS Code Extension" are not indented properly; adjust markdown indentation (e.g., two spaces before sub-items) for correct rendering.
  - Fully Secure 🔐

<a href="https://marketplace.visualstudio.com/items?itemName=iamshreydxv.gitingest"
target="_blank"
rel="noopener noreferrer"
class="text-gray-900 hover:-translate-y-0.5 transition-transform flex items-center gap-1.5">
Copy link

Copilot AI Jun 22, 2025

Choose a reason for hiding this comment

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

Add an aria-label (e.g., aria-label="VS Code Extension") to the anchor tag so screen readers clearly announce the link purpose.

Suggested change
class="text-gray-900 hover:-translate-y-0.5 transition-transform flex items-center gap-1.5">
class="text-gray-900 hover:-translate-y-0.5 transition-transform flex items-center gap-1.5"
aria-label="VS Code Extension">

Copilot uses AI. Check for mistakes.
}
</style>
</defs>
<title>1</title>
Copy link

Copilot AI Jun 22, 2025

Choose a reason for hiding this comment

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

Use a descriptive <title> inside the SVG (e.g., VS Code Extension Icon) instead of a generic "1" for better screen reader support.

Suggested change
<title>1</title>
<title>Navbar Icon</title>

Copilot uses AI. Check for mistakes.
rel="noopener noreferrer"
class="text-gray-900 hover:-translate-y-0.5 transition-transform flex items-center gap-1.5">
<div class="flex items-center">
<svg xmlns="http://www.w3.org/2000/svg"
Copy link

Copilot AI Jun 22, 2025

Choose a reason for hiding this comment

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

The inline SVG is very long and duplicates markup; consider extracting it into a separate partial or using a shared icon component to improve readability and reuse.

Copilot uses AI. Check for mistakes.
@filipchristiansen filipchristiansen changed the title New Feature: VS Code Extension for Gitingest Integration feat: VSCode Extension for Gitingest Integration Jun 22, 2025
@cyclotruc
Copy link
Member

We're discussing with @ShreyPurohit in order to properly migrate his extension into the up and coming github org for gitingest related projects

In the meantime I'll make this a Draft

@cyclotruc cyclotruc marked this pull request as draft June 30, 2025 22:52
@github-actions
Copy link

This pull request has merge conflicts that must be resolved before it can be merged.

@ShreyPurohit
Copy link
Contributor Author

Hi Gitingest team,

Over the past months, I have dedicated significant time — often alongside my full-time job — to complete the tasks aligned with your project’s goals. My intent was to contribute meaningfully and enhance the functionality and reach of Gitingest, without expecting direct personal gain.

However, after repeated follow-ups and months of waiting, I still haven’t received a clear timeline or decision regarding my contribution. Transparency and timely communication are important to me, and without these, it’s difficult to continue in good faith.

I am happy to wait up to the next two weeks for a concrete update. If there’s no decision within that time frame, I will proceed to withdraw my changes and retain them independently. This is not a step I take lightly, but I believe it is the right one under the circumstances.

@ShreyPurohit ShreyPurohit deleted the VsCode-Extension branch September 20, 2025 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rebase needed 🚧 suggestion New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants