Skip to content

Docs/fix grammar spelling#6502

Closed
Ansita20 wants to merge 5 commits intopipe-cd:masterfrom
Ansita20:docs/fix-grammar-spelling
Closed

Docs/fix grammar spelling#6502
Ansita20 wants to merge 5 commits intopipe-cd:masterfrom
Ansita20:docs/fix-grammar-spelling

Conversation

@Ansita20
Copy link
Copy Markdown
Contributor

@Ansita20 Ansita20 commented Feb 2, 2026

Fixes #6460
This PR improves documentation quality by correcting spelling and grammatical errors across multiple doc files.

@Ansita20
Fix links in CONTRIBUTING.md
4ca2d34 
Signed-off-by: ansita20 <ansitasingh20@gmail.com>
@Ansita20
Report event watcher failures
3d7ec14 
Signed-off-by: ansita20 <ansitasingh20@gmail.com>
@Ansita20
docs: fix grammar and spelling errors in v1 documentation
cb01a54 
- Fix spelling: explaination → explanation
- Fix spelling: configuaration → configuration
- Fix grammar: that files → those files
- Fix grammar: the a piped → a piped
- Fix grammar: how install → how to install
- Fix grammar: Please checkout → Please check out
- Fix phrase: community built → community-built

Signed-off-by: ansita20 <ansitasingh20@gmail.com>
Copilot AI review requested due to automatic review settings February 2, 2026 19:20
@Ansita20 Ansita20 requested review from a team as code owners February 2, 2026 19:20
Copilot AI reviewed Feb 2, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

rahulshendre
rahulshendre reviewed Feb 3, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@rahulshendre rahulshendre left a comment

Choose a reason for hiding this comment

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

Hi @Ansita20, thanks for this.
We already have a PR (#6463) that covers these spelling changes and inconsistencies as part of a broader cleanup of the v1 docs.

@Ansita20
Copy link
Copy Markdown
Contributor Author

Ansita20 commented Feb 3, 2026

Hi @Ansita20, thanks for this. We already have a PR (#6463) that covers these spelling changes and inconsistencies as part of a broader cleanup of the v1 docs.

Yaa I did see that but there were more grammar and spelling mistakes still there in the files. So I thought it would be good to make changes.

eeshaanSA
eeshaanSA requested changes Feb 7, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@eeshaanSA eeshaanSA left a comment

Choose a reason for hiding this comment

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

Please review your changes before opening a PR. You have made changes to the event-watcher code in a documentation fix PR.

Comment thread pkg/app/piped/eventwatcher/eventwatcher.go
Copy link
Copy Markdown
Contributor

@eeshaanSA eeshaanSA Feb 7, 2026

Choose a reason for hiding this comment

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

?

Copy link
Copy Markdown
Contributor

@eeshaanSA eeshaanSA Feb 7, 2026

Choose a reason for hiding this comment

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

Please revert.

Comment thread CONTRIBUTING.md
Copy link
Copy Markdown
Contributor

@eeshaanSA eeshaanSA Feb 7, 2026

Choose a reason for hiding this comment

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

This is being tackled in a different PR. Please revert

Copy link
Copy Markdown
Contributor Author

@Ansita20 Ansita20 Feb 7, 2026

Choose a reason for hiding this comment

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

Sorry for the mistake and the delay in the update. I have reverted the changes now.

Ansita20 and others added 2 commits February 7, 2026 19:51
@Ansita20
Reverting the errors
4557c0a 
Signed-off-by: ansita20 <ansitasingh20@gmail.com>
@Ansita20
Merge branch 'master' into docs/fix-grammar-spelling
7a0ec0a 
Signed-off-by: Ansita Singh <146367346+Ansita20@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 10, 2026

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Mar 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 17, 2026

This PR was closed because it has been stalled for 7 days with no activity. Feel free to reopen if still applicable.

@github-actions github-actions Bot closed this Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@eeshaanSA eeshaanSA eeshaanSA requested changes

@khanhtc1202 khanhtc1202 Awaiting requested review from khanhtc1202 khanhtc1202 is a code owner automatically assigned from pipe-cd/pipecd-approvers-docs

@Warashi Warashi Awaiting requested review from Warashi Warashi is a code owner automatically assigned from pipe-cd/pipecd-approvers

@ffjlabo ffjlabo Awaiting requested review from ffjlabo ffjlabo is a code owner automatically assigned from pipe-cd/pipecd-approvers

@t-kikuc t-kikuc Awaiting requested review from t-kikuc t-kikuc is a code owner automatically assigned from pipe-cd/pipecd-approvers

+1 more reviewer

@rahulshendre rahulshendre rahulshendre left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Docs: Improve grammar across v1 docs

4 participants

@Ansita20 @eeshaanSA @rahulshendre