|
|
|
# Wrap review guidelines
|
|
|
|
|
|
|
|
In order to get a package in the Wrap database it must be reviewed and
|
|
|
|
accepted by someone with admin rights. Here is a list of items to
|
|
|
|
check in the review. If some item is not met it does not mean that the
|
|
|
|
package is rejected. What should be done will be determined on a
|
|
|
|
case-by-case basis. Similarly meeting all these requirements does not
|
|
|
|
guarantee that the package will get accepted. Use common sense.
|
|
|
|
|
|
|
|
## Checklist
|
|
|
|
|
|
|
|
Reviewer: copy-paste this to MR discussion box and tick all boxes that apply.
|
|
|
|
|
|
|
|
- [ ] project() has version string
|
|
|
|
- [ ] project() has license string
|
|
|
|
- [ ] if new project, master has tagged commit as only commit
|
|
|
|
- [ ] if new branch, it is branched from master
|
|
|
|
- [ ] contains a readme.txt
|
|
|
|
- [ ] contains an upstream.wrap file
|
|
|
|
- [ ] download link points to authoritative upstream location
|
|
|
|
- [ ] wrap repository contains only build system files
|
|
|
|
- [ ] merge request is pointed to correct target branch (not master)
|
|
|
|
- [ ] wrap works
|
|
|
|
- [ ] repo does not have useless top level directory (i.e. libfoobar-1.0.0)
|