diff options
author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2018-11-13 09:03:10 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-11-13 09:03:10 -0800 |
commit | dd42c1dcffde9628285b4a4ee6d92bca01f3436c (patch) | |
tree | fb0dc9132bb7e1278a5c44ea4ca027f651820f0d /CONTRIBUTING.md | |
parent | 50d3199334a4cc834029bff8d7e256348daf19be (diff) | |
parent | 86d1196f9b30149fa937f7c6680d951c5e4ac997 (diff) | |
download | podman-dd42c1dcffde9628285b4a4ee6d92bca01f3436c.tar.gz podman-dd42c1dcffde9628285b4a4ee6d92bca01f3436c.tar.bz2 podman-dd42c1dcffde9628285b4a4ee6d92bca01f3436c.zip |
Merge pull request #1804 from cevich/lint_verify_image
Standardized container image for gofmt and lint
Diffstat (limited to 'CONTRIBUTING.md')
-rw-r--r-- | CONTRIBUTING.md | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index c4e208894..8e921dcf3 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -180,6 +180,27 @@ Use your real name (sorry, no pseudonyms or anonymous contributions.) If you set your `user.name` and `user.email` git configs, you can sign your commit automatically with `git commit -s`. +### Go Format and lint + +All code changes must pass ``make validate`` and ``make lint``, as +executed in a standard container. The container image for this +purpose is provided at: ``quay.io/libpod/gate:latest``. However, +for changes to the image itself, it may also be built locally +from the repository root, with the command: + +``` +sudo podman build -t quay.io/libpod/gate:latest -f contrib/gate/Dockerfile . +``` + +The container executes 'make' by default, on a copy of the repository. +This avoids changing or leaving build artifacts in your working directory. +Execution does not require any special permissions from the host. However, +the repository root must be bind-mounted into the container at +'/usr/src/libpod'. For example, running `make lint` is done (from +the repository root) with the command: + +``sudo podman run -it --rm -v $PWD:/usr/src/libpod:z quay.io/libpod/gate:latest lint`` + ### Integration Tests Our primary means of performing integration testing for libpod is with the |