Skip to content

Fix #529: Add Missing Requirements to Installation Instructions#530

Open
milelime wants to merge 1 commit into
oppia:developfrom
milelime:fix-pip-install-list
Open

Fix #529: Add Missing Requirements to Installation Instructions#530
milelime wants to merge 1 commit into
oppia:developfrom
milelime:fix-pip-install-list

Conversation

@milelime

@milelime milelime commented Apr 1, 2026

Copy link
Copy Markdown

Overview

This PR fixes the pip install section of the installation instructions #529. Trying to complete the installation of the development server without this edit requires the user to comb through the error logs and manually install the packages. Now, users can simply enter the revised pip install command and immediately proceed to the build step.

Changes

  • Add several requirements to the list of installation commands.

Motivation

This will hopefully make the build process easier for future contributors. This was a frustrating part of getting the developer environment built.

@rohanmalhotracodes

Copy link
Copy Markdown
Contributor

Hey @milelime, I am quite sure we don't have to manually install packages, can you raise a discussion in the main oppia repository about the issue you faced and me and @HardikGoyal2003 might be able to find a fix. Thanks for contributing to oppia.

@rohanmalhotracodes

Copy link
Copy Markdown
Contributor

Also since you might be looking to contribute, just a pointer, the GitHub docs say that to automatically link a issue to a PR write fixes #issue_number instead of mentioning it like you have done in the first line, although there is nothing wrong with it, it just makes it easy for the maintainer to refer to the corresponding issue and get the relevant context.

@milelime

Copy link
Copy Markdown
Author

Also since you might be looking to contribute, just a pointer, the GitHub docs say that to automatically link a issue to a PR write fixes #issue_number instead of mentioning it like you have done in the first line, although there is nothing wrong with it, it just makes it easy for the maintainer to refer to the corresponding issue and get the relevant context.

No problem. Will do so in the future.

@milelime

Copy link
Copy Markdown
Author

Hey @milelime, I am quite sure we don't have to manually install packages, can you raise a discussion in the main oppia repository about the issue you faced and me and @HardikGoyal2003 might be able to find a fix. Thanks for contributing to oppia.

Will do.

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.

2 participants