free
/
searx_ynh
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity
searx_ynh/pull_request_template.md

26 lines
1.1 KiB
Markdown
Raw Normal View History

PR template ## Problem - *The template is hardly used by contributors unless they're part of Apps group.* ## Solution - *Add an automatic template for all PR* ## PR Status Could be reviewed and tested. ## Validation --- *Minor decision* - [ ] **Code review** : - [ ] **Approval (LGTM)** : - [ ] **Approval (LGTM)** : When the PR is mark as ready to merge, you have to wait for 3 days before really merge it.
2018-05-23 11:51:07 +02:00
## Problem
- *Description of why you made this PR*
## Solution
- *And how you fix that*
## PR Status
Replace work by code.
2018-05-23 17:53:30 +02:00
- [ ] Code finished.
Add checkboxes for status
2018-05-23 12:45:32 +02:00
- [ ] Tested with Package_check.
- [ ] Fix or enhancement tested.
- [ ] Upgrade from last version tested.
- [ ] Can be reviewed and tested.
PR template ## Problem - *The template is hardly used by contributors unless they're part of Apps group.* ## Solution - *Add an automatic template for all PR* ## PR Status Could be reviewed and tested. ## Validation --- *Minor decision* - [ ] **Code review** : - [ ] **Approval (LGTM)** : - [ ] **Approval (LGTM)** : When the PR is mark as ready to merge, you have to wait for 3 days before really merge it.
2018-05-23 11:51:07 +02:00
## Validation
---
*Minor decision*
- **Upgrade previous version** :
- [ ] **Code review** :
- [ ] **Approval (LGTM)** :
- [ ] **Approval (LGTM)** :
- **CI succeeded** :
[![Build Status](https://ci-apps-dev.yunohost.org/jenkins/job/searx_ynh%20-BRANCH-%20(Official)/badge/icon)](https://ci-apps-dev.yunohost.org/jenkins/job/searx_ynh%20-BRANCH-%20(Official)/) *Please replace '-BRANCH-' in this link for a PR from a local branch.*
or
[![Build Status](https://ci-apps-dev.yunohost.org/jenkins/job/searx_ynh%20PR-NUM-%20(Official_fork)/badge/icon)](https://ci-apps-dev.yunohost.org/jenkins/job/searx_ynh%20PR-NUM-%20(Official_fork)/) *Replace '-NUM-' by the PR number in this link for a PR from a forked repository.*
Typo fix
2018-05-23 13:40:38 +02:00
When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.