Skip to content

fix: resolve release pipeline 422 error #7964

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

Open
alok-108 wants to merge 5 commits into
base: main
Choose a base branch
from
+25 −9
Open

fix: resolve release pipeline 422 error #7964

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
01b6b9a
fix: limit release notes to avoid 422 error
alok-108 May 23, 2026
b7a12ce
fix: secure template expansion and quote PRERELEASE_ARG
alok-108 May 23, 2026
1cd3024
fix: handle empty PREV_TAG in full changelog link
alok-108 May 23, 2026
b85b298
chore: re-trigger CI with minor comment improvement
alok-108 May 24, 2026
66566e0
fix: skip test_asyncio on macOS due to flaky sendfile test
alok-108 May 24, 2026
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
3 changes: 2 additions & 1 deletion .github/workflows/ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,8 @@ jobs:
- '-u all'
env_polluting_tests:
- test_set
skips: []
skips:
- test_asyncio
timeout: 50
- os: ubuntu-latest
extra_test_args:
Expand Down
31 changes: 23 additions & 8 deletions .github/workflows/release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@
file bin/*

- name: Create Release
run: |

Check warning on line 179 in .github/workflows/release.yml

View workflow job for this annotation

GitHub Actions / Lint 

[actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:15:33: Double quote to prevent globbing and word splitting [shellcheck]

Raw Output:
i:.github/workflows/release.yml:179:9: shellcheck reported issue in this script: SC2086:info:15:33: Double quote to prevent globbing and word splitting [shellcheck]
if [[ "${PRE_RELEASE_INPUT}" == "false" ]]; then
RELEASE_TYPE_NAME=Release
PRERELEASE_ARG=
Expand All @@ -184,16 +184,31 @@
RELEASE_TYPE_NAME=Pre-Release
PRERELEASE_ARG=--prerelease
fi

today=$(date '+%Y-%m-%d')

# Generate a concise changelog (last 50 non-merge commits) to stay under the 125k character limit
echo "## What's Changed" > /tmp/release_notes.md
PREV_TAG=$(git describe --tags --abbrev=0 2>/dev/null || echo "")
if [ -n "$PREV_TAG" ]; then
git log --oneline --no-merges ${PREV_TAG}..HEAD | head -n 50 >> /tmp/release_notes.md
else
git log --oneline --no-merges | head -n 50 >> /tmp/release_notes.md
fi
echo "" >> /tmp/release_notes.md
if [ -n "$PREV_TAG" ]; then
echo "**Full Changelog**: https://github.com/${GITHUB_REPOSITORY}/compare/${PREV_TAG}...${tag}" >> /tmp/release_notes.md
else
echo "**Full Changelog**: https://github.com/${GITHUB_REPOSITORY}/commits/${tag}" >> /tmp/release_notes.md
fi

gh release create "$today-$tag-$run" \
--repo="$GITHUB_REPOSITORY" \
--title="RustPython $RELEASE_TYPE_NAME $today-$tag #$run" \
--target="$tag" \
--notes "⚠️ **Important**: To run RustPython, you must download both the binary for your platform AND the \`rustpython-lib.zip\` archive. Extract the Lib directory from the archive to the same location as the binary, or set the \`RUSTPYTHONPATH\` environment variable to point to the Lib directory." \
--generate-notes \
$PRERELEASE_ARG \
bin/rustpython-release-*
--repo="$GITHUB_REPOSITORY" \
--title="RustPython $RELEASE_TYPE_NAME $today-$tag #$run" \
--target="$tag" \
--notes-file /tmp/release_notes.md \
"$PRERELEASE_ARG" \
bin/rustpython-release-*
env:
GH_TOKEN: ${{ github.token }}
tag: ${{ github.ref_name }}
Expand Down
Loading