Skip to content

docs: fix list grammar in CONTRIBUTING.md tooling section - #961

Open
amir-rezaei wants to merge 2 commits into
ClickHouse:mainfrom
amir-rezaei:docs/fix-contributing-list-grammar
Open

docs: fix list grammar in CONTRIBUTING.md tooling section#961
amir-rezaei wants to merge 2 commits into
ClickHouse:mainfrom
amir-rezaei:docs/fix-contributing-list-grammar

Conversation

@amir-rezaei

Copy link
Copy Markdown

Description

This PR fixes minor list grammar in CONTRIBUTING.md.

Details

Removed both when listing three items and added Oxford comma (supports Node.js, Web environments, and edge runtimes).

Copilot AI review requested due to automatic review settings July 23, 2026 20:31
@amir-rezaei
amir-rezaei requested a review from mshustov as a code owner July 23, 2026 20:31

Copilot AI 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.

Pull request overview

This PR makes a small grammar/style correction in CONTRIBUTING.md by removing an incorrect “both” when listing three items and adding an Oxford comma for clarity.

Changes:

  • Adjusted the “Tooling” paragraph to correctly list three supported environments without “both”.
  • Added an Oxford comma in the environment list for improved readability.

Copilot AI review requested due to automatic review settings August 4, 2026 19:16
@mshustov
mshustov enabled auto-merge (squash) August 4, 2026 19:16
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ mshustov
❌ ferkans-amir
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI 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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

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.

5 participants