From 0d3b526c4d5aa38848a851f959a6856257a8e8c0 Mon Sep 17 00:00:00 2001 From: Jussi Pakkanen Date: Fri, 26 Mar 2021 21:23:04 +0200 Subject: Added some wrap review docs. [skip ci] --- docs/markdown/Adding-new-projects-to-wrapdb.md | 14 +++++++++++++- docs/markdown/Wrap-review-guidelines.md | 26 ++++++++++++++++++++++++++ 2 files changed, 39 insertions(+), 1 deletion(-) diff --git a/docs/markdown/Adding-new-projects-to-wrapdb.md b/docs/markdown/Adding-new-projects-to-wrapdb.md index a1d2486..661ad6c 100644 --- a/docs/markdown/Adding-new-projects-to-wrapdb.md +++ b/docs/markdown/Adding-new-projects-to-wrapdb.md @@ -201,6 +201,14 @@ git commit -a --amend git push --force ``` +### Request a new release version to an existing repository + +Adding new releases to an existing repo is straightforward. All you +need to do is to follow the rules discussed above but when you create +the merge request, file it against the master branch. The repository +reviewer will create the necessary branch and retarget your merge +request accordingly. + ## What is done by WrapDB maintainers [mesonwrap tools](Wrap-maintainer-tools.md) must be used for the tasks @@ -246,4 +254,8 @@ system has native support for Git subprojects. ## Reviewing wraps -See [Wrap review guidelines](Wrap-review-guidelines.md). +See [Wrap review guidelines](Wrap-review-guidelines.md), especially +the part about tooling that you can use to check for the most common +problems yourself. Fixing all issues reported by the tool is a +mandatory requirement for getting your MR accepted, so doing this +proactively makes the review process smoother. diff --git a/docs/markdown/Wrap-review-guidelines.md b/docs/markdown/Wrap-review-guidelines.md index c81a8fa..702e881 100644 --- a/docs/markdown/Wrap-review-guidelines.md +++ b/docs/markdown/Wrap-review-guidelines.md @@ -26,6 +26,32 @@ $ cat ~/.config/mesonwrap.ini github_token = ``` +## Setting up the rewiew tooling + +The [Mesonwrap](https://github.com/mesonbuild/mesonwrap/) repository +contains review tooling. It is used to do the actual review, but +submitters can also use it to check their MRs. All issues reported by +the tool must be fixed, so using the tool can speed up the review +process considerably. + +The tool has some dependencies that are not available in all Linux +distributions. Thus using a Python +[Virtualenv](https://virtualenv.pypa.io/en/stable/). The tool can be +installed with the following commands. + +``` +git clone https://github.com/mesonbuild/mesonwrap.git +cd mesonwrap +virtualenv venv +bash +source venv/bin/activate +pip install -r requirements.txt +# You may need to install Meson and Ninja here as well depending on your setup +python3 setup.py install +mesonwrap review +exit +``` + ## Reviewing code ``` -- cgit v1.1