Skip to content

Fix update-contributors workflow by using a workflow-specific requirements file - #101

Merged
larsvilhuber merged 1 commit into
developmentfrom
copilot/create-workflow-specific-requirements
Sep 1, 2026
Merged

larsvilhuber merged 1 commit into
developmentfrom
copilot/create-workflow-specific-requirements

Conversation

Copilot AI commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor

The Update Contributors List workflow fails during dependency installation: tools/requirements.txt pins PyYAML==6.0, which cannot build from source on Python 3.12.

Collecting PyYAML==6.0 (from -r tools/requirements.txt (line 1))
  ...
  AttributeError: 'build_ext' object has no attribute 'cython_sources'
ERROR: Failed to build 'PyYAML' when getting requirements to build wheel

tools/requirements.txt is consumed by offline processes, so its pins are left alone.

Changes

  • .github/workflows/requirements.txt (new): workflow-only dependency set, containing just requests>=2.32.4. tools/update_contributors.py imports only requests; PyYAML was never needed here.
  • .github/workflows/update-contributors.yml: install from the new file instead of tools/requirements.txt.
- name: Install dependencies
  run: pip install -r .github/workflows/requirements.txt

Follows the existing precedent of docs/requirements.txt for context-specific dependency sets.

Co-authored-by: larsvilhuber <7022712+larsvilhuber@users.noreply.github.com>
Copilot AI changed the title Fix update-contributors workflow: use workflow-specific requirements file Fix update-contributors workflow by using a workflow-specific requirements file Sep 1, 2026
Copilot AI requested a review from larsvilhuber September 1, 2026 06:48
@larsvilhuber
larsvilhuber marked this pull request as ready for review September 1, 2026 07:10
@larsvilhuber
larsvilhuber merged commit 35f41b6 into development Sep 1, 2026
@larsvilhuber
larsvilhuber deleted the copilot/create-workflow-specific-requirements branch September 1, 2026 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants