Skip to content

Conversation

@Excelsius314
Copy link

Add python cmake package template with support for the creation of a virtual environment

@Excelsius314 Excelsius314 self-assigned this May 16, 2025
@Excelsius314 Excelsius314 requested a review from Joschi3 May 16, 2025 14:11
@Excelsius314 Excelsius314 force-pushed the feature/python-cmake-template branch from 3d874da to 00bb906 Compare August 20, 2025 10:43
@Excelsius314
Copy link
Author

Copy link
Member

@Joschi3 Joschi3 left a comment

Choose a reason for hiding this comment

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

Please add tests for the new package type in the CI tests

parser.add_argument("--use-local-templates", action="store_true", help="Use locally installed templates instead of remotely pulling most recent ones")

# parser.add_argument("--template", type=str, default=None, choices=["cpp_pkg", "msgs_pkg", "python_pkg", "ci"], required=True, help="Template")
parser.add_argument("--template", type=str, default=None, choices=["cpp_pkg", "msgs_pkg", "python_pkg", "ci"], required=True, help="Template")
Copy link
Member

Choose a reason for hiding this comment

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

Why did you uncomment this? In any case, your package type is missing in the choices list


| Parameter | Type | Description |
| --- | --- | --- |
| | | |
Copy link
Member

Choose a reason for hiding this comment

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

Maybe add a general section here on how the venv creation and update work.

Excelsius314 and others added 30 commits September 14, 2025 17:38
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