Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Shorter PR template #1429

Merged
merged 2 commits into from
May 21, 2024
Merged

Shorter PR template #1429

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
aa51c42
Shorter PR template
akshayka May 21, 2024
19a78b4
edit
akshayka May 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 12 additions & 25 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,42 +1,29 @@
# Pull Request Template

## 📝 Summary

<!--
Provide a brief summary of what this pull request is addressing. Describe the changes made and the main issue fixed or feature implemented. Keep it concise and informative.
-->

## 🔍 Description of Changes
Provide a concise summary of what this pull request is addressing.

<!--
Detail the specific changes made in this pull request. Explain the problem addressed and how it was resolved. If applicable, provide before and after comparisons or any relevant details to help reviewers understand the changes easily.
If this PR fixes any issues, list them here by number (e.g., Fixes #123).
-->

## 🛠️ Additional Information

## 🔍 Description of Changes

<!--
Include any additional information or context that might be helpful for reviewers. This could include additional dependencies, screenshots, performance improvements, or any other relevant information.
Detail the specific changes made in this pull request. Explain the problem addressed and how it was resolved. If applicable, provide before and after comparisons, screenshots, or any relevant details to help reviewers understand the changes easily.
-->

## 📋 Checklist Before Submitting
## 📋 Checklist

- [ ] I have read the [contributor guidelines](../CONTRIBUTING.md) and the Pull Request section.
- [ ] This PR fixes an issue (If applicable, specify issue number #).
- [ ] This change was discussed or approved through an issue or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable).
- [ ] I have added necessary tests for the changes made.
- [ ] I have read the [contributor guidelines](../CONTRIBUTING.md).
- [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://discord.gg/JE7nhX6mD8), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable).
- [ ] I have added tests for the changes made.
- [ ] I have run the code and verified that it works as expected.

## 🧩 Related Issues
## 📜 Reviewers

<!--
If this PR fixes any issues, list them here by number (e.g., Fixes #123).
-->

## 📜 Who Can Review?

<!--
Tag potential reviewers from the community or maintainers who might be interested in reviewing this pull request. This helps expedite the review process and ensures that the PR receives timely attention.
-->
Tag potential reviewers from the community or maintainers who might be interested in reviewing this pull request.

<!-- Your PR will be reviewed more quickly if you can figure out the right person to tag with @ -->
Your PR will be reviewed more quickly if you can figure out the right person to tag with @ -->
@akshayka OR @mscolnick
Loading