Skip to content

Commit b0eb938

Browse files
Add PR template for new PR's (#287)
1 parent bab6299 commit b0eb938

1 file changed

Lines changed: 50 additions & 0 deletions

File tree

.github/pull_request_template.md

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
### 📝 Description
2+
<!-- A short summary of what this PR does. -->
3+
<!-- Include any relevant context or background information. -->
4+
5+
---
6+
7+
### 🛠️ Changes Made
8+
<!-- - Key changes (e.g., added feature X, refactored Y, fixed Z) -->
9+
10+
---
11+
12+
### ✅ Checklist
13+
- [ ] I have tested the changes in this PR
14+
- [ ] I confirmed whether my changes require a change in documentation and if so, I created another PR in MLRun for the relevant documentation.
15+
- [ ] I confirmed whether my changes require a changes in QA tests, for example: credentials changes, resources naming change and if so, I updated the relevant Jira ticket for QA.
16+
- [ ] I increased the Chart version in `charts/mlrun-ce/Chart.yaml`.
17+
- [ ] I confirmed that the installation works both on a local Docker Desktop environment and on a real cluster when using the required [prerequisites](https://docs.mlrun.org/en/stable/install-mlrun-ce/kubernetes-install.html#prerequisites).
18+
- [ ] If installation issues were found, I updated the relevant Jira ticket with the issue and steps to reproduce, or updated the prerequisites documentation if the issue is related to missing or outdated prerequisites.
19+
- [ ] If needed, update https://github.com/mlrun/ce/blob/development/charts/mlrun-ce/README.md with the relevant installation instructions and version Matrix.
20+
- [ ] If needed, update the following values files for multi namespace support:
21+
- [ ] [Admin values](https://github.com/mlrun/ce/blob/development/charts/mlrun-ce/admin_installation_values.yaml)
22+
- [ ] [User values Node Port](https://github.com/mlrun/ce/blob/development/charts/mlrun-ce/non_admin_installation_values.yaml)
23+
- [ ] [User values ClusterIP](https://github.com/mlrun/ce/blob/development/charts/mlrun-ce/non_admin_cluster_ip_installation_values.yaml)
24+
25+
---
26+
27+
### 🧪 Testing
28+
<!-- - How it was tested (unit tests, manual, integration) -->
29+
<!-- - Any special cases covered. -->
30+
31+
---
32+
33+
### 🔗 References
34+
- Ticket link:
35+
- External links:
36+
- Design docs links (Optional):
37+
---
38+
39+
### 🚨 Breaking Changes?
40+
41+
- [ ] Yes (explain below)
42+
- [ ] No
43+
44+
<!-- If yes, describe what needs to be changed downstream: -->
45+
46+
---
47+
48+
### 🔍️ Additional Notes
49+
<!-- Anything else reviewers should know (follow-up tasks, known issues, affected areas etc.). -->
50+
<!-- ### 📸 Screenshots / Logs -->

0 commit comments

Comments
 (0)