Skip to content

ci: corrected content pushed to python-release #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
susek555 merged 1 commit into from
Dec 19, 2025
Merged

ci: corrected content pushed to python-release #7

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 13 additions & 7 deletions .github/workflows/integration-tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -212,29 +212,35 @@ jobs:
uses: actions/download-artifact@v4
with:
name: real-binary
path: python/teams_lib_pzsp2_z1/bin/
path: /tmp/real-binary

- name: Prepare Client Directory
run: |
chmod +x python/teams_lib_pzsp2_z1/bin/*

mkdir python_release
cp -r python/* python_release/

mkdir -p python_release/.github/workflows
# binaries
TARGET_BIN_DIR="python_release/teams_lib_pzsp2_z1/bin/"
mkdir -p $TARGET_BIN_DIR
find /tmp/real-binary -type f -exec cp {} $TARGET_BIN_DIR \;
chmod +x $TARGET_BIN_DIR/*

# publish workflow
mkdir -p python_release/.github/workflows
cp .github/workflows/publish.yml python_release/.github/workflows/

# license file
if [ -f "LICENSE" ]; then
cp LICENSE python_release/
fi

# clean up unneeded files
rm -rf python_release/tests
find python_release -type d -name "__pycache__" -exec rm -rf {} +

Copilot AI Dec 19, 2025

Copy link

Choose a reason for hiding this comment

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

The find command with -exec rm -rf {} + can fail when removing nested pycache directories because it may attempt to access a parent directory after its child has already been deleted. This can cause the command to exit with an error. Consider using -prune flag or pipe to xargs with proper handling, or use -delete flag instead. For example: find python_release -type d -name "__pycache__" -prune -exec rm -rf {} \; or add || true to ignore errors if that's acceptable.

Suggested change
find python_release -type d -name "__pycache__" -exec rm -rf {} +
find python_release -type d -name "__pycache__" -prune -exec rm -rf {} \;

Copilot uses AI. Check for mistakes.
rm -rf python_release/.venv
rm -rf python_release/uv.lock

ls -R python_release/teams_lib_pzsp2_z1/bin
ls -R python_release

- name: Commit and Push to Release Branch
run: |
Expand All @@ -243,9 +249,9 @@ jobs:
git checkout --orphan python-release

git rm -rf .
cp -r python_release/* .
cp -a python_release/. .
rm -rf python_release
git add .

git add .
git commit -m "Release build: ${{ github.sha }}"
git push -f origin python-release