Skip to content

Include repo-level wheelhouse/ in GitHub Pages deployment#20

Merged
Creeper19472 merged 3 commits intomainfrom
copilot/add-check-whl-file-deploy
Feb 24, 2026
Merged

Include repo-level wheelhouse/ in GitHub Pages deployment#20
Creeper19472 merged 3 commits intomainfrom
copilot/add-check-whl-file-deploy

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 24, 2026

The deploy_index job only picked up wheels built during CI. Pre-built .whl files committed to the project root wheelhouse/ directory were silently ignored and never deployed to Pages.

Changes

  • .github/workflows/wheels.yml — Added Include pre-built wheels from wheelhouse step in deploy_index, between "Organize wheels" and "Generate PyPI index". Copies any .whl files from the repo-root wheelhouse/ into wheels/ before index generation. Gracefully no-ops when the directory is absent or empty.
- name: Include pre-built wheels from wheelhouse
  run: |
    if [ -d wheelhouse ] && [ -n "$(find wheelhouse -maxdepth 1 -name '*.whl' -print -quit)" ]; then
      echo "Found pre-built wheels in wheelhouse/:"
      find wheelhouse -maxdepth 1 -name '*.whl' -exec ls -lh {} \;
      find wheelhouse -maxdepth 1 -name '*.whl' -exec cp {} wheels/ \;
    else
      echo "No pre-built wheels found in wheelhouse/"
    fi

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…ment

Co-authored-by: Creeper19472 <38857196+Creeper19472@users.noreply.github.com>
Copilot AI changed the title [WIP] Add check for .whl file in wheelhouse folder during deployment Include repo-level wheelhouse/ in GitHub Pages deployment Feb 24, 2026
@Creeper19472 Creeper19472 marked this pull request as ready for review February 24, 2026 17:00
Copilot AI review requested due to automatic review settings February 24, 2026 17:00
@Creeper19472 Creeper19472 merged commit 678657c into main Feb 24, 2026
@Creeper19472 Creeper19472 deleted the copilot/add-check-whl-file-deploy branch February 24, 2026 17:01
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the GitHub Pages deployment to include pre-built wheel files from the repository's wheelhouse/ directory, which were previously ignored during the deployment process.

Changes:

  • Modified .gitignore to allow committing wheel files in the wheelhouse/ directory
  • Added a workflow step to copy pre-built wheels from wheelhouse/ into the deployment directory before index generation

Reviewed changes

Copilot reviewed 1 out of 4 changed files in this pull request and generated 1 comment.

File Description
.gitignore Removes exclusion of wheelhouse/ directory and *.whl files to allow committing pre-built wheels
.github/workflows/wheels.yml Adds step to include pre-built wheels from wheelhouse/ directory in the Pages deployment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +799 to +802
if [ -d wheelhouse ] && [ -n "$(find wheelhouse -maxdepth 1 -name '*.whl' -print -quit)" ]; then
echo "Found pre-built wheels in wheelhouse/:"
find wheelhouse -maxdepth 1 -name '*.whl' -exec ls -lh {} \;
find wheelhouse -maxdepth 1 -name '*.whl' -exec cp {} wheels/ \;
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The workflow runs find twice over the same directory structure. Capture the list once and reuse it for both operations to avoid redundant filesystem traversal.

Suggested change
if [ -d wheelhouse ] && [ -n "$(find wheelhouse -maxdepth 1 -name '*.whl' -print -quit)" ]; then
echo "Found pre-built wheels in wheelhouse/:"
find wheelhouse -maxdepth 1 -name '*.whl' -exec ls -lh {} \;
find wheelhouse -maxdepth 1 -name '*.whl' -exec cp {} wheels/ \;
if [ -d wheelhouse ]; then
mapfile -t wheel_files < <(find wheelhouse -maxdepth 1 -name '*.whl' -type f)
if [ "${#wheel_files[@]}" -gt 0 ]; then
echo "Found pre-built wheels in wheelhouse/:"
ls -lh "${wheel_files[@]}"
cp "${wheel_files[@]}" wheels/
else
echo "No pre-built wheels found in wheelhouse/"
fi

Copilot uses AI. Check for mistakes.
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.

3 participants