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

Feature python package main script #90

Merged
merged 5 commits into from
Aug 30, 2020

Conversation

chaps
Copy link
Contributor

@chaps chaps commented Aug 20, 2020

Adds support for installing as python package:
enables installing from project root with: pip install ./
enables installing from remote source: e.g. pip install -U git+ssh://git@github.com/chaps/OrganiseDesktop.git@feature-python-package-main-script
enables CLI command organise_desktop after installing
enables calling GUI application from module : python -m organise_desktop after installing

  • Adds init.py and main.py to treat organise_desktop
    dir as a python package.
  • Moves setup.py to the root of the project, enables pip install ./ from
    the root of the project.
  • Edits Cli.py import statements to import local modules from relative import.
  • Wraps main function in Cli.py to enable console script.
  • organise_desktop command added to entry_points for cli usage.
  • Wraps Clean.py main function and imports it in main module

- Adds __init__.py and __main__.py to treat `organise_desktop`
dir as a python package.
- Moves setup.py to the root of the project, enables `pip install ./` from
the root of the project.
- Edits Cli.py import statements to import local modules from relative import.
- Wraps main function in Cli.py to enable console script.
- organise_desktop command added to entry_points for cli usage.
@KokoseiJ
Copy link
Collaborator

Reviewed. Perfect. I'm accepting this.

@KokoseiJ KokoseiJ merged commit 8aed58f into blavejr:master Aug 30, 2020
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