Commit a21a4bca authored by Mathieu Coupé's avatar Mathieu Coupé Committed by Pierre Smeyers
Browse files

fix: use default config when env variable is set but empty

parent ea2dde71
Loading
Loading
Loading
Loading
+4 −4
Original line number Diff line number Diff line
@@ -26,7 +26,7 @@ def generate_parser():
    )
    parser.add_argument(
        "--api-url",
        default=os.getenv("GITLAB_API", "https://gitlab.com/api/v4"),
        default=os.getenv("GITLAB_API") or "https://gitlab.com/api/v4",
        help="GitLab API url",
    )
    parser.add_argument(
@@ -82,7 +82,7 @@ def generate_parser():
    parser.add_argument(
        "--pipeline-deletion-limit",
        type=int,
        default=int(os.getenv("PIPELINES_DELETION_LIMIT", "0")),
        default=int(os.getenv("PIPELINES_DELETION_LIMIT") or "0"),
        help="maximum number of pipelines deleted per project",
    )
    parser.add_argument(
@@ -106,13 +106,13 @@ def generate_parser():
    parser.add_argument(
        "--pipelines-keep-per-mr",
        type=int,
        default=int(os.getenv("PIPELINES_KEEP_PER_MR", "1")),
        default=int(os.getenv("PIPELINES_KEEP_PER_MR") or "1"),
        help="number of pipelines to keep per merge request (⚠ MR MUST still exist)",
    )
    parser.add_argument(
        "--pipelines-delete-older-than",
        type=int,
        default=int(os.getenv("PIPELINES_DELETE_OLDER_THAN", "30")),
        default=int(os.getenv("PIPELINES_DELETE_OLDER_THAN") or "30"),
        help="max age (in days) after which pipelines are deleted (unless they are kept by a keep rule)",
    )
    return parser
+82 −82

File changed.

Preview size limit exceeded, changes collapsed.