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

Improve the Toolkit's README #137

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

luismartingarcia
Copy link
Contributor

This patch modifies the ACI toolkit's README file so it contains a bit more information and looks cleaner. Since this is often the first thing that newcomers see, I think it's beneficial to work a bit on the messaging and the look&feel. It also adds a "Contributing" section encouraging users to clone the repository and extend it to their needs.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 1d8364f on luismartingarcia:improve-readme into 700c386 on datacenter:master.

@mateja82
Copy link

Maybe add how exactly to add the environment variables (PYTHONPATH) with an example.

@robphoenix
Copy link

Is this PR dead? If so I'd be interested in reusing it to open a fresh PR. I think there's some good info here that the project would benefit from having in its README.

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.

4 participants