# 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)
