Skip to content

Setting Up Git: Unify ChromeOS instructions into Linux section #29627

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

miryarik
Copy link

Because

The ChromeOS Git installation instructions were isolated in their own section and used an out-dated workaround, which created redundancy and confusion for users. Since ChromeOS now comes with Linux Development Environment and APT can be used to install packages as one would on Debian, it makes sense to update the instructions and unify them under the existing Linux section for correctness and relevance.

This PR

  • Moved ChromeOS-specific Git installation instructions into the Linux section
  • Added a tip callout for ChromeOS users at the start of the Linux section
  • Removed the standalone ChromeOS section to reduce duplication
  • Included both recommended and legacy options for ChromeOS users within the Linux section

Issue

Closes #28232

Additional Information

Includes reference link to Google Support for Turning On Linux on ChromeOS.

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

ChromOS now comes with Linux Development Environment which includes APT.
Installation can be done using APT as on a regular Linux system.
Instructions for Linux can now be used by ChromeOS users as well.
Guide on turning on Linux in ChromeOS is linked as reference.
Article on installation using source is linked as legacy approach for older Chromebooks.
Instructions for installation on Linux cover the installation process for ChromeOS as well.
Added a short note for ChromeOS users for clarity and ease.
Guide to enable Linux on ChromeOS and legacy installation using source are linked for reference.
@github-actions github-actions bot added the Content: Foundations Involves the Foundations content label Apr 16, 2025
Copy link
Member

@xandora xandora left a comment

Choose a reason for hiding this comment

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

@miryarik After reviewing this PR, I think it might be better if we keep the ChromeOS dropdown section, but just update the method for installing Git.

We also don't really want to suggest that we provide support for older Chromebooks that don't have access to the Linux Dev Environment. When we brought Chromebooks on, it was purely because the Linux Dev Env was releasing soon and we had it as a beta. We didn't provide support for any Chromebook before that, and it's not something we want to start including.

If you can have another go to address this changes, we should be in a good place to get this merged.

@miryarik
Copy link
Author

@xandora sure thing, i'll have another go. I'll rewrite the linux instructions and update the chromeos ones a bit. both in their separate sections as they are right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content: Foundations Involves the Foundations content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting up Git: Replace DigitalOcean installation instructions
2 participants