On Wed, Jul 22, 2020 at 10:21:43 +0200, Ján Tomko wrote:
On a Thursday in 2020, Pavel Hrdina wrote:
> Having limit set to 100 is in most cases perfect but sometimes there can
> be a larger series that will have more than 100 patches and it will make
> the check-dco job fail.
>
> Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com>
> ---
> .gitlab-ci.yml | 2 +-
> ci/cirrus/build.yml | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 702198ec8e4..c997dc6df25 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -1,5 +1,5 @@
> variables:
> - GIT_DEPTH: 100
> + GIT_DEPTH: 1000
>
> stages:
> - sanity_checks
> diff --git a/ci/cirrus/build.yml b/ci/cirrus/build.yml
> index 893e13d7241..49e90b6d67b 100644
> --- a/ci/cirrus/build.yml
> +++ b/ci/cirrus/build.yml
> @@ -14,7 +14,7 @@ build_task:
> install_script:
> - @INSTALL_COMMAND@ @PKGS@
> clone_script:
> - - git clone --depth 100 "$CI_REPOSITORY_URL" .
> + - git clone --depth 1000 "$CI_REPOSITORY_URL" .
Does cirrus run the dco-check too or is this just for consistency?
> - git fetch origin "$CI_COMMIT_REF_NAME"
> - git reset --hard "$CI_COMMIT_SHA"
> build_script:
Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
Do we really want to merge this in the end?