Skip to content

[README] Edits syntax throughout ReadMe for better clarity/flow - #15283

Closed
stephvelazquez wants to merge 1 commit into
apache:masterfrom
stephvelazquez:master
Closed

[README] Edits syntax throughout ReadMe for better clarity/flow#15283
stephvelazquez wants to merge 1 commit into
apache:masterfrom
stephvelazquez:master

Conversation

@stephvelazquez

Copy link
Copy Markdown

What changes were proposed in this pull request?

(Please fill in changes proposed in this fix)
Grammatical edits to the ReadMe to improve legibility

How was this patch tested?

(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)

(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

No test necessary

@AmplabJenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

Comment threadREADME.md
build/mvn -DskipTests clean package

(You do not need to do this if you downloaded a pre-built package.)
(You do not need to do this if you have downloaded a pre-built package.)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't agree with most of these changes. This was grammatical. The rest is just alternate wording which I don't think is meaningfully better.

@asfgitasfgit closed this in 5e9f32dOct 6, 2016
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 22, 2025
## What changes were proposed in this pull request?
This PR proposes to close some stale PRs and ones suggested to be closed by committer(s) or obviously inappropriate PRs (e.g. branch to branch).
Closesapache#13458Closesapache#15278Closesapache#15294Closesapache#15339Closesapache#15283
## How was this patch tested?
N/A
Author: hyukjinkwon <gurwls223@gmail.com>
Closesapache#15356 from HyukjinKwon/closing-prs.
Sign up for freeto 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.

3 participants

@stephvelazquez@AmplabJenkins@srowen