Skip to content

Update pull_request_template.md#1032

Merged
VisargD merged 2 commits into
Portkey-AI:mainfrom
code-crusher:feat/update-pr-template
Apr 22, 2025
Merged

Update pull_request_template.md#1032
VisargD merged 2 commits into
Portkey-AI:mainfrom
code-crusher:feat/update-pr-template

Conversation

@code-crusher
Copy link
Copy Markdown
Contributor

@code-crusher code-crusher commented Apr 11, 2025

Code Quality type: docs

Title: Update pull_request_template.md

Description:

🔄 What Changed

  • Updated the .github/pull_request_template.md file
  • Introduced new sections: Description, Motivation, Type of Change, How Has This Been Tested?, Screenshots, Checklist, Related Issues
  • Added checklists to guide contributors

🔍 Impact of the Change

  • Standardizes PR information
  • Improves review efficiency
  • Promotes self-review and testing

📁 Total Files Changed

  • 1 file: .github/pull_request_template.md (32 additions, 9 deletions)

🧪 Test Added

  • N/A

🔒 Security Vulnerabilities

  • None

Motivation:

  • To enhance the quality and consistency of Pull Request submissions and streamline the review process

Related Issues:

  • N/A

Quality Recommendations

  1. Consider adding a link to the project's contribution guidelines within the template for easy reference

  2. Add a section for 'Deployment Considerations' to help reviewers understand any deployment impacts

  3. Include a section for 'Dependencies Added/Modified' to track changes to project dependencies

@VisargD VisargD merged commit 48f0a89 into Portkey-AI:main Apr 22, 2025
1 check 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.

3 participants