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

Update installation instructions, troubleshooting instructions, and CI check failure instructions. #431

Merged
merged 3 commits into from
Jan 8, 2025

Conversation

seanlip
Copy link
Member

@seanlip seanlip commented Jan 6, 2025

  1. Update installation instructions to say, make sure you have Java 11 on your machine ("java --version" should show Java 11). We were previously recommending Java 8.

  2. Update https://github.com/oppia/oppia/wiki/If-CI-checks-fail-on-your-PR to escalate if multiple runs fail.

  3. Add [BUG]: Unable to start dev-server using python -m scripts.start. Subprocess exited with error oppia#21617 to wiki troubleshooting page for MacOS.

@seanlip
Copy link
Member Author

seanlip commented Jan 6, 2025

@Helper2020 Please could you review this PR? Thanks!

Copy link
Collaborator

@Helper2020 Helper2020 left a comment

Choose a reason for hiding this comment

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

@seanlip Review done.


```
sudo apt-get install python3-pip python3-setuptools curl openjdk-8-jre git python3-dev python3-yaml python3-matplotlib unzip libbz2-dev
sudo apt-get install curl git unzip openjdk-11-jre
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think running sudo update-alternatives --config java command should be put here in case the dev has multiple versions and wants to make it default.

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

@Helper2020 Helper2020 Jan 8, 2025

Choose a reason for hiding this comment

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

@seanlip Maybe the below note should be moved here since its right after java 11 installed? "Check that Java version 11+ is installed. You can use the command java -version to do this. If you need to change the default binary for the java command, use sudo update-alternatives --config java."

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks @Helper2020, done! PTAL. I also merged the first two steps.

@@ -111,7 +111,7 @@ Then run `pip install future`.

### No Java

If running `which java` on the terminal does not return any output, you do not have java installed. You can install it by running `sudo apt install openjdk-7-jre-headless`. Note that this command might vary for your local machine.
If running `which java` on the terminal does not return any output, you do not have java installed. You can install it by running `sudo apt install openjdk-11-jre`. Note that this command might vary for your local machine.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same as other comment.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, updated. PTAL.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, just saw the LGTM. Thanks, I'll go ahead and merge!

@seanlip
Copy link
Member Author

seanlip commented Jan 7, 2025

@Helper2020 I addressed all comments, PTAL. Thanks!

Copy link
Collaborator

@Helper2020 Helper2020 left a comment

Choose a reason for hiding this comment

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

LGTM


```
sudo apt-get install python3-pip python3-setuptools curl openjdk-8-jre git python3-dev python3-yaml python3-matplotlib unzip libbz2-dev
sudo apt-get install curl git unzip openjdk-11-jre
Copy link
Collaborator

@Helper2020 Helper2020 Jan 8, 2025

Choose a reason for hiding this comment

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

@seanlip Maybe the below note should be moved here since its right after java 11 installed? "Check that Java version 11+ is installed. You can use the command java -version to do this. If you need to change the default binary for the java command, use sudo update-alternatives --config java."

@seanlip seanlip merged commit f1c980a into develop Jan 8, 2025
3 checks passed
@seanlip seanlip deleted the housekeeping branch January 8, 2025 18:32
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