docs icon indicating copy to clipboard operation
docs copied to clipboard

Content improvement: Add missing Enterprise Server versioning to Copilot IDE article

Open aymanaboghonim opened this issue 7 months ago • 12 comments

Summary

The asking-github-copilot-questions-in-your-ide.md article is missing Enterprise Server (GHES) version coverage, only showing content for GitHub Enterprise Cloud (GHEC).

Location

  • File: content/copilot/using-github-copilot/copilot-chat/asking-github-copilot-questions-in-your-ide.md
  • Problem: Missing GHES versioning blocks
  • Impact: Enterprise Server users cannot access relevant content sections

Current problem

The article contains GHEC-only versioning blocks without corresponding GHES coverage:

Missing GHES blocks:

  • Line 32-35: GHEC-only content about GitHub Copilot Chat availability
  • Line 89-92: GHEC-only content about organizational policies

Enterprise Server users see incomplete information or missing sections.

Proposed solution

  1. Add GHES versioning blocks - Include appropriate {% ifversion ghes %} blocks for Enterprise Server content
  2. Verify content accuracy - Ensure GHES-specific content reflects actual Enterprise Server capabilities
  3. Test rendering - Verify the article renders correctly for both GHEC and GHES versions
  4. Follow versioning patterns - Align with established GitHub Docs versioning standards

Additional context

Reference: Identified during PR #38720 review by @Sharra-writes Related: Part of broader versioning consistency improvement initiative

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

aymanaboghonim avatar Jun 09 '25 19:06 aymanaboghonim

@aymanaboghonim Like the other issue, no one needs to go all-in on this, it would just be nice for the versioning to display correctly for right now. And to update the links you identified without breaking everything.

Sharra-writes avatar Jun 09 '25 20:06 Sharra-writes

NOTE: Per maintainer feedback on issue #38825, the GitHub Docs team has plans for a new simplified versioning system on the horizon, and we should focus only on minimal fixes rather than broader versioning campaigns.

@Sharra-writes - Should this Enterprise versioning issue be closed since it's part of the broader campaign approach you mentioned isn't needed right now? Or would you like this kept open for the future simplified versioning system implementation?

Happy to close this and focus only on the minimal quickstart.md fixes in #38825. 🙏

aymanaboghonim avatar Jun 09 '25 20:06 aymanaboghonim

@aymanaboghonim I would like to see those links updated, which may require addressing the versioning. And if we have to address it anyway, and it's not too complicated, it makes sense to make sure it's displaying properly. If we can sneak in and get those links without addressing it, that would be fine, too.

Sharra-writes avatar Jun 09 '25 22:06 Sharra-writes

Hey @Sharra-writes, what is up ?? hoping you are doing well 🥇 It is more than welcome to help if possible

aymanaboghonim avatar Jun 17 '25 10:06 aymanaboghonim

@aymanaboghonim Sorry, the versioning for this is just really complex, so I've been working on checking it in between other work, but it's taking a while. In general, things in this repo move fairly slowly unless it's a simple fix. Things have to be double checked, SMEs frequently have to be called in, and it's a surprisingly small team for the number of docs we have. I appreciate your patience. I haven't forgotten and it hasn't even gotten lost in my pile of work, it's just slow and involved.

Sharra-writes avatar Jun 17 '25 17:06 Sharra-writes

Ic3 aware I been hacked

Fool256 avatar Oct 12 '25 05:10 Fool256

name:%20PostgreSQL%20service%20example%0Aon%3A%20push%0A%0Ajobs%3A%0A%20%20%23%20Label%20of%20the%20container%20job%0A%20%20container-job:%0A%20%20%20%20%23%20Containers%20must%20run%20in%20Linux%20based%20operating%20systems%0A%20%20%20%20runs-on:%20ubuntu-latest%0A%20%20%20%20%23%20Docker%20Hub%20image%20that%20%60container-job%60%20executes%20in%0A%20%20%20%20container:%20node:20-bookworm-slim%0A%0A%20%20%20%20%23%20Service%20containers%20to%20run%20with%20%60container-job%60%0A%20%20%20%20services:%0A%20%20%20%20%20%20%23%20Label%20used%20to%20access%20the%20service%20container%0A%20%20%20%20%20%20postgres:%0A%20%20%20%20%20%20%20%20%23%20Docker%20Hub%20image%0A%20%20%20%20%20%20%20%20image:%20postgres%0A%20%20%20%20%20%20%20%20%23%20Provide%20the%20password%20for%20postgres%0A%20%20%20%20%20%20%20%20env:%0A%20%20%20%20%20%20%20%20%20%20POSTGRES_PASSWORD:%20postgres%0A%20%20%20%20%20%20%20%20%23%20Set%20health%20checks%20to%20wait%20until%20postgres%20has%20started%0A%20%20%20%20%20%20%20%20options:%20%3E-%0A%20%20%20%20%20%20%20%20%20%20--health-cmd%20pg_isready%0A%20%20%20%20%20%20%20%20%20%20--health-interval%2010s%0A%20%20%20%20%20%20%20%20%20%20--health-timeout%205s%0A%20%20%20%20%20%20%20%20%20%20--health-retries%205%0A%0A%20%20%20%20steps:%0A%20%20%20%20%20%20%23%20Downloads%20a%20copy%20of%20the%20code%20in%20your%20repository%20before%20running%20CI%20tests%0A%20%20%20%20%20%20-%20name:%20Check%20out%20repository%20code%0A%20%20%20%20%20%20%20%20uses:%20actions/checkout@v5%0A%0A%20%20%20%20%20%20%23%20Performs%20a%20clean%20installation%20of%20all%20dependencies%20in%20the%20%60package.json%60%20file%0A%20%20%20%20%20%20%23%20For%20more%20information,%20see%20https://docs.npmjs.com/cli/ci.html%0A%20%20%20%20%20%20-%20name:%20Install%20dependencies%0A%20%20%20%20%20%20%20%20run:%20npm%20ci%0A%0A%20%20%20%20%20%20-%20name:%20Connect%20to%20PostgreSQL%0A%20%20%20%20%20%20%20%20%23%20Runs%20a%20script%20that%20creates%20a%20PostgreSQL%20table,%20populates%0A%20%20%20%20%20%20%20%20%23%20the%20table%20with%20data,%20and%20then%20retrieves%20the%20data.%0A%20%20%20%20%20%20%20%20run:%20node%20client.js%0A%20%20%20%20%20%20%20%20%23%20Environment%20variables%20used%20by%20the%20%60client.js%60%20script%20to%20create%20a%20new%20PostgreSQL%20table.%0A%20%20%20%20%20%20%20%20env:%0A%20%20%20%20%20%20%20%20%20%20%23%20The%20hostname%20used%20to%20communicate%20with%20the%20PostgreSQL%20service%20container%0A%20%20%20%20%20%20%20%20%20%20POSTGRES_HOST:%20postgres%0A%20%20%20%20%20%20%20%20%20%20%23%20The%20default%20PostgreSQL%20port%0A%20%20%20%20%20%20%20%20%20%20POSTGRES_PORT:%205432%0A

anildazzlee avatar Oct 17 '25 13:10 anildazzlee