Skip to content

Update README.md - #820

Open
isarkis wants to merge 2 commits into
mainfrom
isarkis-patch-14
Open

Update README.md#820
isarkis wants to merge 2 commits into
mainfrom
isarkis-patch-14

Conversation

@isarkis

@isarkis isarkis commented Jan 13, 2026

Copy link
Copy Markdown
Member

No description provided.

SortaCookie and others added 2 commits January 13, 2026 14:00
Change-Id: Ie0ee0d86efca90dfcf2fe02e46ab8a92ddbec62e
@isarkis
isarkis requested a review from a team as a code owner January 13, 2026 22:31
@isarkis
isarkis requested review from kaidokert and removed request for a team January 13, 2026 22:31

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request adds a blank line to the README.md file. My review includes a suggestion to remove this line, as it creates an unnecessary paragraph break that affects the readability of the instructions for checking out the source code.

Comment thread README.md
The project's web site is https://www.chromium.org.

To check out the source code locally, don't use `git clone`! Instead,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This added blank line creates a paragraph break, which makes the sentence feel disconnected and harder to read. It's better to keep the instruction as a single, continuous paragraph by removing this empty line.

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