Skip to content

Conversation

ShahanaFarooqui
Copy link
Collaborator

The Publish distribution stage was failing because it executed the update-pyln-versions script from within the WORKDIR, which created an invalid context. To resolve this, we have decoupled the process, separating the updating of version state into its own step that runs from the root directory before the publish operation.

Changelog-None.

@madelinevibes madelinevibes added this to the v25.12 milestone Sep 3, 2025
@ShahanaFarooqui ShahanaFarooqui requested review from sangbida and removed request for madelinevibes September 3, 2025 05:58
@rustyrussell
Copy link
Contributor

This is great, BUT:

The Publish distribution stage was failing because it executed the update-pyln-versions script from within the WORKDIR, which created an invalid context. To resolve this, we have decoupled the process, separating the updating of version state into its own step that runs from the root directory before the publish operation.

This NEEDS to go in the commit message. Everything in GitHub will vanish and be lost, only the git commit contents will remain. And I will not be able to tell why this change was made from the commit message:

 ci: Add a new step to update pyln versions out of WORKDIR

Changelog-None.

?

Copy link
Contributor

@rustyrussell rustyrussell left a comment

Choose a reason for hiding this comment

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

Needs content in the commit msg!

@ShahanaFarooqui ShahanaFarooqui force-pushed the fix-pypi-update-versions branch from f953d32 to 79d6646 Compare September 4, 2025 05:06
The Publish distribution stage was failing because it executed the update-pyln-versions script from within the WORKDIR, which created an invalid context. To resolve this, we have decoupled the process, separating the updating of version state into its own step that runs from the root directory before the publish operation.

Changelog-None.
@ShahanaFarooqui ShahanaFarooqui force-pushed the fix-pypi-update-versions branch from 79d6646 to 7b65df3 Compare September 4, 2025 05:10
@ShahanaFarooqui
Copy link
Collaborator Author

I added the description in the commit but changelog should still be none as it just fixes the CI and does not directly impact other users/developers.

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