Skip to content

Commit only pyproject.toml and uv.lock from the idfkit bump - #30

Merged
samuelduchesne merged 1 commit into
mainfrom
fix/bump-add-paths
Oct 14, 2026
Merged

samuelduchesne merged 1 commit into
mainfrom
fix/bump-add-paths

Conversation

@samuelduchesne

Copy link
Copy Markdown
Contributor

1. Defect

bump-idfkit.yml checks idfkit-conformance out into governance/, inside the working tree, and called peter-evans/create-pull-request@v7 without add-paths. The action commits every change in the tree, so the nested checkout was committed as a gitlink: governance, mode 160000, pointing at idfkit-conformance 2b96631, with no .gitmodules entry. Any later actions/checkout of a commit holding it fails with fatal: No url found for submodule path 'governance' in .gitmodules.

The open bump pull request #29 carries that gitlink. It has been removed from #29's branch separately, so #29 can merge; this change stops the next bump from adding it. idfkit-plugin merged the same defect into main and has been failing every checkout since (idfkit/idfkit-plugin#23).

2. Change

create-pull-request receives add-paths listing pyproject.toml and uv.lock, the two files the workflow writes (the dependency with [tool.idfkit.library] level, then uv lock).

The bump workflow checks idfkit-conformance out into governance/ and let create-pull-request commit the whole tree, so the checkout was committed as a gitlink with no .gitmodules entry (#29). The workflow now names the two files it writes.
@github-actions

Copy link
Copy Markdown
Contributor

Docs preview for this PR is available at:
https://developers.idfkit.com/pr-preview/pr-30/

github-actions Bot added a commit that referenced this pull request Oct 14, 2026
@samuelduchesne
samuelduchesne merged commit f91e756 into main Oct 14, 2026
7 checks passed
@samuelduchesne
samuelduchesne deleted the fix/bump-add-paths branch October 14, 2026 15:08
github-actions Bot added a commit that referenced this pull request Oct 14, 2026
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.

1 participant