-
Notifications
You must be signed in to change notification settings - Fork 153
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Shorter PR template * edit
- Loading branch information
Showing
1 changed file
with
12 additions
and
25 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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 |