this post was submitted on 18 Nov 2024
15 points (67.4% liked)

Programming

17668 readers
142 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities [email protected]



founded 2 years ago
MODERATORS
 

Hey guys! I built an AI powered file organizer! This was my first "big" Python project!

you are viewing a single comment's thread
view the rest of the comments
[–] [email protected] 16 points 1 month ago* (last edited 1 month ago) (1 children)

Some feedback:

  • On white background the text next to the logo is not visible
  • Add screenshots in the README, it's a GUI app
  • Requirements.txts for dependency management is the old way, read about pyproject.toml you can merge them a single easy to read and edit file
  • "Install the dependencies" means nothing to a non-python developer. Direct users to install your project via pipx, that's modern and secure way of installing a python application with dependencies for non developers. Publish it to pypi for even easier installation.
  • Add a notice that currently it's windows only os.path.join(os.environ["APPDATA"], "Tagify", "config.yaml") will fail on *nix systems. Use pathlib.Path instead of os.path. Use pathlib, I see on a lot more places it would make your life much easier.
  • I have a feeling that the file icons are not your work. If you copied them from somewhere make sure their license is compatible, and add an acknowledgement.

Keep up the work, it seems like a nice project!

[–] [email protected] 4 points 1 month ago (1 children)

Thanks! I fixed the file icon licensing! However, I'm not sure will pipx help. I already provide a binary Inno Setup installer. Any suggestions how to port it to Linux? I dual boot - so it would be very useful for me.

[–] [email protected] 2 points 1 month ago

Python is installed by default on all linux and mac systems, so it's just one more command to install pipx. From there just pipx install tagify. You don't need an installer, just specify the build tools in pyproject.toml: https://packaging.python.org/en/latest/specifications/pyproject-toml/#declaring-build-system-dependencies-the-build-system-table e.g. with setuptools: https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html

If you publish to pypi it will build the wheel files when you publish a version. That's the easiest way I know.

Innosetup is windows only. On linux you don't need such a thing.