Skip to content

Conversation

@gdlg
Copy link
Contributor

@gdlg gdlg commented Aug 8, 2025

Summary

Remove the concept of project from the CLI

Part of #1809

How to test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have added the description of my changes into CHANGELOG.​
  • I have updated the documentation accordingly

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below).
# Copyright (C) 2025 Intel Corporation
#
# SPDX-License-Identifier: MIT

@gdlg gdlg requested a review from AlbertvanHouten August 19, 2025 07:36
@AlbertvanHouten
Copy link
Contributor

There are still quite a few references to project throughout the code base such as the error messages in src/datumaro/components/errors.py and the project builder in src/datumaro/components/project.py.

src/datumaro/project.py can also be deleted

@gdlg
Copy link
Contributor Author

gdlg commented Aug 19, 2025

There are still quite a few references to project throughout the code base such as the error messages in src/datumaro/components/errors.py and the project builder in src/datumaro/components/project.py.

src/datumaro/project.py can also be deleted

That’s intended. Since this PR is already quite large, I wanted to focus on the CLI and not change the API. I will delete those in the next PR.

@AlbertvanHouten
Copy link
Contributor

Ah apologies, I didn't notice you specifically targeted the CLI.

@gdlg gdlg merged commit 970504b into develop Aug 19, 2025
15 checks passed
@gdlg gdlg deleted the gppayend/project-cli branch August 28, 2025 09:56
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