Skip to content
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.

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

Solve poetry requirement for README.md #21

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

OlivierCloudar
Copy link
Contributor

Since poetry 2.0 the install command requires the README.md to be present. Also the --no-root is required as we are not really installing the "current project" but only the dependencies.

@rondefreitas
Copy link

@cuchi Facing the same issue which has blocked our Github builds as well

jcfr added a commit to Slicer/slicer.org that referenced this pull request Jan 8, 2025
As of 2025-01-08, the latest version of `cuchi/jinja2-action` broke due
to a breaking change introduced in poetry 2.0.
See cuchi/jinja2-action#21

This fork was created to provide the fix originally contributed by @OlivierCloudar,
ensuring it is associated with a GitHub organization rather than an
individual user account.
See /~https://github.com/Slicer/jinja2-action
jcfr added a commit to Slicer/slicer.org that referenced this pull request Jan 8, 2025
As of 2025-01-08, the latest version of `cuchi/jinja2-action` broke due
to a breaking change introduced in poetry 2.0.
See cuchi/jinja2-action#21

This fork was created to provide the fix originally contributed by @OlivierCloudar,
ensuring it is associated with a GitHub organization rather than an
individual user account.
See /~https://github.com/Slicer/jinja2-action
@cuchi cuchi merged commit 70f3757 into cuchi:master Jan 8, 2025
@cuchi
Copy link
Owner

cuchi commented Jan 8, 2025

Thanks!

@dmytrolevin-fn
Copy link

I can't see new release on the main page of this repo

@rondefreitas
Copy link

@cuchi can you please publish this as a 1.2.3 release so it updates on Marketplace? Thanks!

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.

5 participants