docker-hub icon indicating copy to clipboard operation
docker-hub copied to clipboard

Is this project will maintain continue

Open tinymahua opened this issue 2 years ago • 4 comments

tinymahua avatar Aug 03 '23 06:08 tinymahua

Yes, this will be maintained.

amalfra avatar Aug 03 '23 06:08 amalfra

I see the last commit time is 2022 year, so I asked this.

Recently I found 2 problem when I test this repo and pip package.

  1. After [pip install ], it create [src] directory in site-packages, src is a no meaning name, import it use import src is not fit.
  2. Modules import in source file are relative mode, I does not think this is a good way.

Base above, i asked it. Fix them need change it's package structure and relative mode. I hope I can join and maintain it. Can you accept this change about package structure and some what.

Yes, this will be maintained.

tinymahua avatar Aug 04 '23 01:08 tinymahua

@tinymahua https://packaging.python.org/en/latest/discussions/src-layout-vs-flat-layout/ is a description of src layouts. It's a standard layout and since we don't want code to be used src layout works best.

amalfra avatar Aug 13 '23 05:08 amalfra

@tinymahua https://packaging.python.org/en/latest/discussions/src-layout-vs-flat-layout/ is a description of src layouts. It's a standard layout and since we don't want code to be used src layout works best.

I agreen src layouts. I mean that it need a main package named docker_hub in src directory. And then set the setup.py target the installed packages.

Now code structure will happen a problem after pip install: In my project, I use docker_hub need do:

from src.libs.xxx import yyy

Normal import statements should above:

from docker_hub.xxxx import yyy

Perhaps I desc not enough before, after a few time, I commit a PR, hope think it, thanks.

tinymahua avatar Aug 24 '23 09:08 tinymahua