Skip to content

Conversation

@alineavila
Copy link
Contributor

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • My contribution adds a new instruction, prompt, or chat mode file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, or chat mode with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.

Description

This PR adds a generic code review instruction file that can be reused across any project using GitHub Copilot.

Key Features

  • Multi-language support (JavaScript, Python, Java, TypeScript)
  • Three priority levels (Critical, Important, Suggestion)
  • Comprehensive coverage: security, performance, architecture, testing
  • Structured comment format template
  • Highly customizable for project-specific needs
  • Based on GitHub Copilot prompt engineering best practices

Type of Contribution

  • New instruction file.
  • New prompt file.
  • New chat mode file.
  • New collection file.
  • Update to existing instruction, prompt, chat mode, or collection.
  • Other (please specify):

Additional Notes

Testing: Tested with Java/Spring Boot microservices and multiple programming languages.

Differentiation: Provides a generic, framework-agnostic approach with educational examples (wrong ❌ vs correct ✅ code) and comprehensive security, performance, and architecture checks.


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

- Generic and reusable code review template
- Multi-language support (JavaScript, Python, Java, TypeScript)
- Based on GitHub Copilot prompt engineering best practices
- Includes security, performance, and architecture checks
- Customizable for any project
- Follows awesome-copilot contribution guidelines
- Generic and reusable code review template
- Multi-language support (JavaScript, Python, Java, TypeScript)
- Based on GitHub Copilot prompt engineering best practices
- Includes security, performance, and architecture checks
- Customizable for any project
- Follows awesome-copilot contribution guidelines
Copilot AI review requested due to automatic review settings November 28, 2025 22:01
Copilot finished reviewing on behalf of alineavila November 28, 2025 22:02
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 comprehensive generic code review instruction file that provides reusable guidelines for GitHub Copilot across any project. The instruction file covers security, performance, testing, architecture, and code quality standards with concrete examples in multiple programming languages (JavaScript, Python, Java, TypeScript). The contribution follows the repository's naming conventions and includes proper README documentation.

Key Changes

  • New instruction file with extensive code review guidelines organized by priority levels (Critical, Important, Suggestion)
  • Multi-language examples demonstrating good and bad practices across common review categories
  • README updated to include the new instruction in alphabetical order with proper install badges

Reviewed changes

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

File Description
instructions/code-review-generic.instructions.md New comprehensive code review instruction file with guidelines covering security, performance, testing, architecture, and code quality standards with concrete examples
docs/README.instructions.md Added entry for the new generic code review instructions in alphabetical order with proper formatting and install links

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@aaronpowell aaronpowell merged commit ddfccc7 into github:main Nov 30, 2025
2 checks passed
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.

2 participants