[docs] Remove Python 2 installation steps
ClosedPublic
Actions

Authored by atul on May 16 2022, 12:47 PM.

Details

Reviewers

Commits

rCOMM4b8c8de82fb2: [docs] Remove Python 2 installation steps

Summary

We were previously including Python2 because it was needed as part of the sqlite3 build process and Python2 was removed from macOS as of 12.3.0

However, the sqlite3 npm package has recently been updated and this issue has been resolved.

Depends on D4055

Test Plan

Remove Python 2
Ensure that Python 2 is uninstalled by running which python and where python (Python 3 is python3)
Run yarn cleaninstall and see if it succeeds

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

atul published this revision for review.May 16 2022, 12:47 PM

atul edited the summary of this revision. (Show Details)

Before landing, can you create two tasks:

One to remove the Python2 requirement (and defaulting python to Python2) from Nix (assign to Jon or Varun)
Another for the same thing, but for Docker (assign to me)

This revision is now accepted and ready to land.May 16 2022, 1:45 PM