Skip to content

Commit 7c19c3d

Browse files
authored
Merge pull request #125 from FormalLanguageConstrainedPathQuerying/dev
Fix GitHub Release creation in the publish workflow
2 parents 5e20af3 + d16fec8 commit 7c19c3d

3 files changed

Lines changed: 21 additions & 1 deletion

File tree

‎.github/workflows/publish.yml‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,10 @@ jobs:
7171
if: startsWith(github.ref, 'refs/tags/v')
7272
runs-on: ubuntu-latest
7373
steps:
74+
# Needed for the GitHub Release step, which extracts the release notes
75+
# from CHANGELOG.md.
76+
- uses: actions/checkout@v7
77+
7478
- name: Download dist
7579
uses: actions/download-artifact@v8
7680
with:

‎tasks/detailed_plan.md‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,3 +146,19 @@ pre-release validation (a tag always points at a merged PR's head).
146146
**Code:** `.github/workflows/publish.yml`
147147
**Tests:** skip — workflow-only; the build step is the check itself
148148
**Docs:** `docs/release.rst` (Package publishing section), `CHANGELOG.md`
149+
150+
### S9: Check out the repo in the publish-pypi job
151+
152+
Added 2026-09-16 at the publish gate. The real tag push published to PyPI
153+
successfully, but "Create GitHub Release" failed with
154+
`awk: fatal: cannot open file 'CHANGELOG.md'`: the `publish-pypi` job never
155+
checked out the repository (it only downloads the dist artifact), so the
156+
changelog was absent from its working directory. Latent bug — the step had
157+
never run before because every earlier attempt died upstream. The v5.0.0
158+
GitHub Release was created manually (same notes + the exact PyPI artifacts as
159+
assets) while this fix lands via PR for future releases.
160+
161+
**Code:** `.github/workflows/publish.yml` (`actions/checkout@v7` in
162+
`publish-pypi`)
163+
**Tests:** skip — workflow-only
164+
**Docs:** none

‎tasks/tasks.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,4 +34,4 @@
3434
- [done] Task 32: Cool. Yes, fix Babel warnings. Analyze rest warnings. E.g. why you suppress aernings refs.citations and ref.footnote? Can we fix it and other warnings? I want to set warings as errors and go to 'no warnings policy'. Without suppresuins if possible. **[USER GUIDANCE]**: "Well. Try to find solution that does not require workarounds or checkers modification. If site o code refactoring required for this way, propose it to me. It may be easier to do smoll refactoring, rather than add workaround. If refactoring improve and simplify site and code, it is better that wotkaround or tooling patching." / "9 approved. Go."
3535
- [done] Task 33: Check developer documantation. Check that all parts of issue https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/76 well-documented, necessary parts added to README. **[USER GUIDANCE]**: "Let use docs as the source of truth, skills as thin pointers. Add this to main pronciples for future skills design and update."
3636
- [done] Task 34: Prepare v5.0.0 release (1/2): analyze the code with respect to the issues https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/122 https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/76 https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/75 https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/74 https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/28 https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/26 https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/16 https://github.com/FormalLanguageConstrainedPathQuerying/CFPQ_Data/issues/2 and add respective references to the Changelog for the resolved ones. **[USER GUIDANCE]**: "Reference partial, no close claim" (issue #2: only the Zheng & Rugina C alias analysis data is in; the Vedurada 'Batch Alias Analysis' data is missing) / "Reference resolved sub-items only" (umbrella issue #28: cite the resolved sub-issues #27, #32, #20, #30, #33, #26 individually; do not claim #28 done).
37-
- [ ] Task 35: Prepare v5.0.0 release (2/2): cut the v5.0.0 release per the release skill (bump_version, PR dev->master, tag on origin/master, push, verify PyPI + GitHub Release). **[USER GUIDANCE]**: "Yes, both tasks. But do not forget to ask me to assist with access to PyPI. Guide me step-by-step. And do not forget that I must review and merge requext to master manually, so, tag + push and PyPI/GitHub release only after I confirm that requies merged. Moreover you should pull master irst to create tag on respective commit." / "Trusted publisher added. Go."
37+
- [done] Task 35: Prepare v5.0.0 release (2/2): cut the v5.0.0 release per the release skill (bump_version, PR dev->master, tag on origin/master, push, verify PyPI + GitHub Release). **[USER GUIDANCE]**: "Yes, both tasks. But do not forget to ask me to assist with access to PyPI. Guide me step-by-step. And do not forget that I must review and merge requext to master manually, so, tag + push and PyPI/GitHub release only after I confirm that requies merged. Moreover you should pull master irst to create tag on respective commit." / "Trusted publisher added. Go."

0 commit comments

Comments
 (0)