From 7bcaabab53c55ae74d4e496c0edf6b643341c224 Mon Sep 17 00:00:00 2001 From: Alexander VanTol Date: Mon, 30 Mar 2026 08:40:58 -0500 Subject: [PATCH 1/3] Simplify CI workflow by removing LintConfig Removed lint configuration steps from CI workflow. --- .github/workflows/ci.yml | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index aeda6be1..7b2395f1 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -23,13 +23,6 @@ jobs: python-version: '3.13' use-cache: true - # this creates linter settings and uploads to an artifact so the configs can be pulled and used across jobs - LintConfig: - name: Get Lint Config - uses: uc-cdis/.github/.github/workflows/lint-create-config.yaml@master - with: - python-module-name: "gen3Dictionary" - RequiredLint: name: Run Required Linters needs: [ LintConfig ] @@ -37,12 +30,3 @@ jobs: with: python-version: '3.13' use-cache: true - - InformationalLint: - name: Run Informational Linters - needs: [ LintConfig ] #TODO Add UnitTest - if: github.ref != 'refs/heads/main' - uses: uc-cdis/.github/.github/workflows/optional_lint_check.yaml@master - with: - python-version: '3.13' - use-cache: true From 9511e36e5dd39a929e5e7465282e7951e3f10607 Mon Sep 17 00:00:00 2001 From: Alexander VanTol Date: Mon, 30 Mar 2026 12:26:26 -0500 Subject: [PATCH 2/3] Remove dependency on LintConfig for RequiredLint job --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 7b2395f1..07931af9 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -25,7 +25,7 @@ jobs: RequiredLint: name: Run Required Linters - needs: [ LintConfig ] + needs: [ ] uses: uc-cdis/.github/.github/workflows/required_lint_check.yaml@master with: python-version: '3.13' From a9994e4d1f307fb7fb6b5117751aa3e373f5a7df Mon Sep 17 00:00:00 2001 From: Alexander VanTol Date: Mon, 30 Mar 2026 12:44:04 -0500 Subject: [PATCH 3/3] fix(pre-commit): use https instead of git for compatibility --- .pre-commit-config.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index e20af678..a0d115ab 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -1,5 +1,5 @@ repos: - - repo: git@github.com:Yelp/detect-secrets + - repo: https://github.com/Yelp/detect-secrets rev: v1.5.0 hooks: - id: detect-secrets