Skip to content

fix: Fix templates - poetry-plugin-export version and camoufox template name#952

Merged
Pijukatel merged 2 commits intomasterfrom
template-issues
Feb 3, 2025
Merged

fix: Fix templates - poetry-plugin-export version and camoufox template name#952
Pijukatel merged 2 commits intomasterfrom
template-issues

Conversation

@Pijukatel
Copy link
Copy Markdown
Collaborator

Description

Add poetry-plugin-export version constraint <1.9 to make it compatible with already constrained version of poetry <2
(poetry-plugin-export>=1.9 works only with poetry version >=2)

Fix incorrect camoufox cookiecutter template name in several template files 'camoufox' -> 'playwright-camoufox'

Issues

@Pijukatel Pijukatel added bug Something isn't working. t-tooling Issues with this label are in the ownership of the tooling team. labels Feb 3, 2025
@github-actions github-actions bot added this to the 107th sprint - Tooling team milestone Feb 3, 2025
@Pijukatel Pijukatel changed the title Fix templates - poetry-plugin-export version and camoufox template name fix: Fix templates - poetry-plugin-export version and camoufox template name Feb 3, 2025
@Pijukatel Pijukatel marked this pull request as ready for review February 3, 2025 07:53
@Pijukatel Pijukatel requested review from janbuchar and vdusek February 3, 2025 07:53
This is optimization, otherwise those files will be automatically fetched on each image start.
Copy link
Copy Markdown
Collaborator

@janbuchar janbuchar left a comment

Choose a reason for hiding this comment

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

LGTM, but please make a follow up issue to update to poetry 2

Copy link
Copy Markdown
Collaborator

@vdusek vdusek left a comment

Choose a reason for hiding this comment

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

lgtm

@Pijukatel
Copy link
Copy Markdown
Collaborator Author

LGTM, but please make a follow up issue to update to poetry 2

#954

@Pijukatel Pijukatel merged commit 7addea6 into master Feb 3, 2025
@Pijukatel Pijukatel deleted the template-issues branch February 3, 2025 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Template issues failing docker image build and error in camoufox template

3 participants