Skip to content

Commit 6fa52e8

Browse files
committed
fix: remove duplicate validate-workflows job from main CI
- Remove validate-workflows job from ci.yml (already exists in separate workflow) - Prevents chicken-and-egg problem where broken CI can't validate itself - Workflow validation should only run in dedicated validate-workflows.yml - Maintains separation of concerns between CI and workflow validation
1 parent 174eda9 commit 6fa52e8

1 file changed

Lines changed: 0 additions & 31 deletions

File tree

.github/workflows/ci.yml

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -75,37 +75,6 @@ jobs:
7575
make quality-check
7676
fi
7777
78-
validate-workflows:
79-
name: Validate Workflows
80-
runs-on: ubuntu-latest
81-
needs: get-config
82-
83-
steps:
84-
- name: Checkout code
85-
uses: actions/checkout@v5
86-
87-
- name: Install yq
88-
uses: mikefarah/yq@master
89-
90-
- name: Set up Python
91-
uses: actions/setup-python@v5
92-
with:
93-
python-version: ${{ needs.get-config.outputs.default-python-version }}
94-
95-
- name: Install UV
96-
uses: astral-sh/setup-uv@v6
97-
with:
98-
enable-cache: true
99-
cache-dependency-glob: "pyproject.toml.template"
100-
101-
- name: Install PyYAML
102-
run: |
103-
uv venv
104-
uv pip install PyYAML
105-
106-
- name: Validate workflows
107-
run: make validate-all
108-
10978
lint-black:
11079
name: Black (Code Formatting)
11180
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)