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

Updates getting-started.rst #179

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

Updates getting-started.rst #179

wants to merge 1 commit into from

Conversation

antoineaws
Copy link

Since the latest brew python is now at 3.9.4, it throws errors when executing pip3 install -r requirements.txt. Hence I added a couple lines for notes in case someone runs into these issues.
Cheers.

to: @airbnb/binaryalert-maintainers
cc:
size: small|medium|large
resolves #

Background

Documentation enhancement to make sure the correct version of python is installed and if not, where to get python 3.6.x.

Changes

Adds 2 notes to help users in case they run into similar issues.

Testing

Ran the additional commands added to the document locally and it passed.

Since the latest brew python is now at 3.9.4, it throws errors when executing pip3 install -r requirements.txt. Hence I added a couple lines for notes in case someone runs into these issues.
Cheers.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.52% when pulling 7b9235d on antawad:patch-1 into a9c0f06 on airbnb:master.

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

2 participants