vcpkg/docs/maintainers/pr-review-checklist.md
nicole mazzuca 0fec1340eb
[vcpkg manifest] Add documentation! ()
* [vcpkg docs] add docs for manifest files

These are just for the maintainer docs,
not user docs.

* [vcpkg] EBNF-ify platform expression parsing

this modifies nothing about what strings are accepted or rejected,
it just moves stuff around.

also adds tests.

* [vcpkg docs] add manifest mode example

* [wip] docs for augustin

also fix tabs

* [vcpkg manifest] switch to using maps for features

* Apply suggestions from code review

* un-experimentize format-manifest

* flesh out the user manifest mode docs

* CRs

* billy CRs

* final personal pass-thru
2020-09-14 15:07:02 -07:00

5.8 KiB

Vcpkg PR Checklist

Revision: 1

Overview

This document provides an annotated checklist which vcpkg team members use to apply the "reviewed" label on incoming pull requests. If a pull request violates any of these points, we may ask contributors to make necessary changes before we can merge the changeset.

Feel free to create an issue or pull request if you feel that this checklist can be improved. Please increment the revision number when modifying the checklist content.

Checklist

You can link any of these checklist items in a GitHub comment by copying the link address attached to each item code.

c000001: No deprecated helper functions are used

See our Maintainer Guidelines and Policies for more information.

c000002: `"port-version"` field is updated

See our Maintainer Guidelines and Policies for more information.

c000003: New ports contain a `"description"` field written in English

A description only one or a few sentences long is helpful. Consider using the library's official description from their README.md or similar if possible. Automatic translations are acceptable and we are happy to clean up translations to English for our contributors.

See our manifest file documentation for more information.

c000004: No unnecessary comments are present in the changeset

See our Maintainer Guidelines and Policies for more information.

c000005: Downloaded archives are versioned if availableTo ensure archive content does not change, archives downloaded preferably have an associated version tag that can be incremented alongside the port's "version".

c000006: New ports pass CI checks for triplets that the library officially supports

To ensure vcpkg ports are of a high quality, we ask that incoming ports support the official platforms for the library in question.

c000007: Patches fix issues that are vcpkg-specific only

If possible, patches to the library source code should be upstreamed to the library's official repository. Opening up a pull request on the library's repository will help to improve the library for everyone, not just vcpkg users.

c000008: New ports download source code from the official source if available

To respect library authors and keep code secure, please have ports download source code from the official source. We may make exceptions if the original source code is not available and there is substantial community interest in maintaining the library in question.

c000009: Ports and port features are named correctly

For user accessibility, we prefer names of ports and port features to be intuitive and close to their counterparts in official sources and other package managers. If you are unsure about the naming of a port or port feature, we recommend checking repology.org, packages.ubuntu.com, or searching for additional information using a search engine. We can also help our contributors with this, so feel free to ask for naming suggestions if you are unsure.

c000010: Library targets are exported when appropriate

To provide users with a seamless build system integration, please be sure to export and provide a means of finding the library targets intended to be used downstream. Targets not meant to be exported should be be marked private and not exported.

c000011: Ports do not use applications which modify the user's system

Ports should uphold vcpkg's contract of not modifying the user's system by avoiding applications which do so. Examples of these applications are sudo, apt, brew, or pip. Please use an alternative to these types of programs wherever possible.

c000012: Ports with system dependencies include an information message during installation

Some ports have library and tool dependencies that do not exist within vcpkg. For these missing dependencies, we ask that contributors add a message to the top of the port's portfile.cmake stating the missing dependencies and how to acquire them. We ask that the message is displayed before any major work is done to ensure that users can "early out" of the installation process as soon as possible in case they are missing the dependency.

Example:

message(
"${PORT} currently requires the following libraries from the system package manager:
    autoconf libtool
These can be installed on Ubuntu systems via sudo apt install autoconf libtool"
)
c000013: Manifest files are used instead of CONTROL files for new ports

Many existing ports use the CONTROL file syntax; while this syntax will be supported for some time to come, new ports should not use these. Any newly added port must use the manifest files.

We also recommend, when significant modifications are made to ports, that one switches to manifest files; however, this is not required. You may find vcpkg format-manifest useful.