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

pre-deploy clean-up #14

Merged
merged 10 commits into from Apr 13, 2018
Merged

pre-deploy clean-up #14

merged 10 commits into from Apr 13, 2018

Conversation

jesteria
Copy link
Member

Added some bootstrapping and installation support, and moved app and bin under src as their own Python packages.

We might want to add to README that the package may be installed with:

python setup.py install

and/or named as a Python install requirement (e.g. in a requirements file).

And that you can then execute the auditor with:

aequitas-audit

And we might add a separate section, expressly for developers, that you can set up your local environment by executing:

./install

Copy link
Collaborator

@abbystvns abbystvns left a comment

Choose a reason for hiding this comment

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

👍 thanks for fixing this!

@saleiro saleiro merged commit bb271c8 into webapp Apr 13, 2018
@saleiro
Copy link
Member

saleiro commented Apr 13, 2018

awesome blossom!

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.

None yet

3 participants