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

Src layout #133

Merged
merged 3 commits into from
Aug 29, 2024
Merged

Src layout #133

merged 3 commits into from
Aug 29, 2024

Conversation

kba
Copy link
Contributor

@kba kba commented Aug 29, 2024

Remove the qurator namespace #112

Switch to src-layout, i.e. all python sources are in src, in the eynollah package.

For users of the command line tool, there is no change.

When using eynollah in python scripts, one just has to remove the qurator. prefix:

-from qurator.eynollah.eynollah import Eynollah
+from eynollah.eynollah import Eynollah

(BTW we could do from .eynollah import Eynollah in __init__.py, then you could do from eynollah import Eynollah which is less typing).

This includes #131, so merging this PR will also merge the other.

I'd recommend merging this to main soon and then to merge main into the active development branches, because there will be some conflicts and the sooner we change this, the less the branches will diverge.

@cneud cneud requested review from vahidrezanezhad and michalbubula and removed request for vahidrezanezhad and michalbubula August 29, 2024 16:19
@cneud cneud merged commit 256a7c3 into main Aug 29, 2024
4 checks passed
@kba kba mentioned this pull request Sep 2, 2024
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.

2 participants