Skip to content

add custom copilot instructions - #64

Closed
faatmalman-eng wants to merge 8 commits into
github-samples:mainfrom
hackathon-blue-frog-5:main
Closed

add custom copilot instructions #64
faatmalman-eng wants to merge 8 commits into
github-samples:mainfrom
hackathon-blue-frog-5:main

Conversation

@faatmalman-eng

Copy link
Copy Markdown

Description

Related Issue

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📚 Documentation update
  • 🧪 Test update
  • 🔧 Refactor (no functional changes)

Changes Made

Testing

Data Layer Changes

  • Ran npm run test:unit - all tests pass
  • Added/updated Vitest tests for data-layer changes
  • Generated a migration (npm run db:generate) for any schema change

Frontend Changes

  • Ran npm run test:e2e - all tests pass
  • Added data-testid attributes to interactive elements
  • Verified build succeeds (npm run build)

Checklist

  • My code follows the project's coding standards
  • I have used explicit TypeScript types for function parameters and return values
  • I have built the UI with Astro components and Tailwind CSS utility classes (dark theme)
  • I have updated documentation (README, instruction files) if needed
  • My changes are focused on a single concern
  • I have written clear commit messages explaining what and why

Additional Notes

Copilot AI lite review requested due to automatic review settings August 26, 2026 12:09

Copilot AI left a comment

Copy link
Copy Markdown

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 Copilot-related learning/resource links to the repository documentation by prepending them to README.md and introducing a new “Useful Docs” Markdown file at the repo root.

Changes:

  • Added Copilot workshop links and embedded screenshots at the top of README.md.
  • Added a new DOCUMENT.S.md file containing a short list of Copilot-related resource links.

Reviewed changes

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

File Description
README.md Prepends Copilot workshop links and screenshots ahead of the project’s main README content.
DOCUMENT.S.md Adds a new repo-root “Useful Docs” page with Copilot-related external links.

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

Comment thread README.md
Comment on lines +1 to +9
Basic workshop url: https://github-samples.github.io/copilot-workshops/vscode/0-prerequisites/
Agentic devops workshop: https://copilot-dev-days.github.io/agentic-workflows-workshop/step.html?step=readme

Free hack:
<img width="1238" height="646" alt="image" src="https://github.com/user-attachments/assets/889908e1-2d72-42c5-be0c-6382d6d4b023" />
<img width="1243" height="646" alt="image" src="https://github.com/user-attachments/assets/7f891afb-1d87-4db1-af38-cefa93edab60" />



Comment thread DOCUMENT.S.md
Comment on lines +1 to +3
# Useful Docs

- Tokens optimization: https://olivomarco.github.io/github-copilot-token-optimization/#comparisons
Introduce getAllCategories/getAllPublishers helpers and extend getAllGames with optional category/publisher filters (AND across groups, OR within a group), with unit test coverage.
New GameFilters.astro component renders accessible checkbox groups and filters the prerendered game grid client-side; GameCard now exposes category/publisher ids for filtering.
Covers filter visibility, single/multi-category OR, publisher filter, combined AND, clear filters, and keyboard interaction.
Update README features section and copilot-instructions repository structure to reflect the new filtering helpers, component, and tests.
Add category and publisher filtering to the game list
@hackathon-blue-frog-5 hackathon-blue-frog-5 closed this by deleting the head repository Aug 28, 2026
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.

5 participants