Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: VSCode Extension for Gitingest Integration #242

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

Draft
ShreyPurohit wants to merge 12 commits into coderamp-labs:main
base: main
Choose a base branch
Loading
from ShreyPurohit:VsCode-Extension

Conversation

Copy link
Contributor

@ShreyPurohit ShreyPurohit commented Mar 19, 2025
edited by filipchristiansen
Loading

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 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
Preview

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
Preview

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
Preview

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
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

ShreyPurohit reacted with thumbs up emoji

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

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

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@cyclotruc cyclotruc Awaiting requested review from cyclotruc

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
rebase needed 🚧 suggestion New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /