From 08d31d4fc74913a7f6998bfda0de9da2d8d05bbd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?louiz=E2=80=99?= Date: Thu, 18 Oct 2018 21:32:51 +0200 Subject: [PATCH] Remove the formatting-check ci build until we decide what we should do with it MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit If we ignore it, it’s just noise. We need to decide: - fix the code to be like yapf wants to be (I’m against this) - change the rules, to make them way more flexible, and then apply them - don’t run yapf at all --- .gitlab-ci.yml | 9 --------- 1 file changed, 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index f1f09443..93129d0d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -86,12 +86,3 @@ pylint-plugins: - pip3 install -r requirements-plugins.txt - python3 setup.py install - pylint -E plugins - -formatting-check: - stage: test - image: python:3 - allow_failure: true - script: - - pip3 install yapf - - yapf -dpr poezio - - "[ -n \"$(yapf -dpr poezio)\" ] && echo 'Formatting check failed, please run yapf' && exit 1 || echo 'Formatting check succeeded'"