Skip to content

Workspace creation module#9

Merged
viktorbeck98 merged 25 commits intomainfrom
workspace
Dec 15, 2025
Merged

Workspace creation module#9
viktorbeck98 merged 25 commits intomainfrom
workspace

Conversation

@annaerdi
Copy link
Copy Markdown
Contributor

@annaerdi annaerdi commented Dec 1, 2025

This PR adds a mate create command to add new parser/detector workspaces.

Given a target directory (--dir) and a name (--name), the command:

  • Creates a Python package <name>/ with __init__.py and <name>.py based on a template.
  • Creates a tests folder with pre-filled test_<name>.py based on a template.
    • The test files in the template_tests folder are runnable tests as part of the library
    • Added tests in tests/test_workspace to ensure the generation and the the generated tests also work
  • Copies project meta files (LICENSE.md, .gitignore, .pre-commit-config.yaml) into the workspace root.
  • Generates pyproject.toml file and a README describing the workspace and how to use it.
    • DetectMateLibrary is added to the main dependencies
    • DetectMateService is added as optional dependency with a description in the README on how to run a component as a service
  • Does not overwrite an existing package directory to avoid accidental data loss.

@annaerdi annaerdi marked this pull request as ready for review December 3, 2025 12:03
@ipmach
Copy link
Copy Markdown
Contributor

ipmach commented Dec 3, 2025

Looks good, lets wait for the Schema refactor #10 and add the new changes here before accepting the merge.

@annaerdi annaerdi marked this pull request as draft December 3, 2025 16:18
@annaerdi annaerdi marked this pull request as ready for review December 5, 2025 13:48
@annaerdi annaerdi requested a review from whotwagner December 5, 2025 14:42
@annaerdi
Copy link
Copy Markdown
Contributor Author

As required by this feature, the service is updated too in this PR.

Copy link
Copy Markdown
Collaborator

@viktorbeck98 viktorbeck98 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@viktorbeck98 viktorbeck98 merged commit fe0fd63 into main Dec 15, 2025
1 check passed
@annaerdi annaerdi deleted the workspace branch December 16, 2025 15:41
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