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

feat: publish as pypi package #17

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nullswan
Copy link
Contributor

@nullswan nullswan commented Oct 6, 2022

Address #18

  • feat: added CD workflow | Will create release/push to pypi on tag.
  • feat: pyproject file
  • refactor: version system
  • refactor: moved scripts assets into the main repository

Instead of doing python main.py which is an anti pattern and not sustainable, you will be able to install using pip and call it like graphw00f ....
You are also able to call it using python -m graphw00f from a local directory.

To setup the local env using poetry, install poetry (pip install poetry) and then do poetry install poetry shell.

@nullswan nullswan mentioned this pull request Oct 6, 2022
@noraj noraj mentioned this pull request Nov 24, 2022
Copy link

@noraj noraj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the refactoring, this is way more cleaner

@omar2535
Copy link

Any further updates on this? Packaging this tool would be nice

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