Skip to content

docs: fix grammar in prefer-const rule description#20883

Closed
monqyClouds wants to merge 2 commits into
eslint:mainfrom
monqyClouds:docs/prefer-const-grammar
Closed

docs: fix grammar in prefer-const rule description#20883
monqyClouds wants to merge 2 commits into
eslint:mainfrom
monqyClouds:docs/prefer-const-grammar

Conversation

@monqyClouds

Copy link
Copy Markdown

Summary

Two small grammar fixes in docs/src/rules/prefer-const.md:

  • "declared using let keyword" → "declared using the let keyword"
  • For the "all" destructuring option: "this rule warns the variables" → "this rule warns for those variables" (parallels the wording used for "any" two lines above)

No content/behavior changes — wording only.

@monqyClouds monqyClouds requested a review from a team as a code owner May 18, 2026 05:01
@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage May 18, 2026
@linux-foundation-easycla

linux-foundation-easycla Bot commented May 18, 2026

Copy link
Copy Markdown

CLA Missing ID

  • ✅ login: monqyClouds / name: monqyClouds (ace410f)
  • ❌ The email address for the commit (a129453) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please visit our EasyCLA portal and chat with our support bot.

@eslint-github-bot eslint-github-bot Bot added the documentation Relates to ESLint's documentation label May 18, 2026
@netlify

netlify Bot commented May 18, 2026

Copy link
Copy Markdown

Deploy Preview for docs-eslint ready!

Name Link
🔨 Latest commit ace410f
🔍 Latest deploy log https://app.netlify.com/projects/docs-eslint/deploys/6a0b82d56e19c20008c57205
😎 Deploy Preview https://deploy-preview-20883--docs-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@lumirlumir lumirlumir left a comment

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.

Thanks for the PR. Could you sign the CLA so we can move forward?

Also, please be sure to review our AI Usage Policy and use our PR template for future contributions.

@lumirlumir lumirlumir moved this from Needs Triage to Triaging in Triage May 18, 2026
@monqyClouds monqyClouds requested a review from lumirlumir May 18, 2026 21:13
@amareshsm

Copy link
Copy Markdown
Member

The CLA is still pending. Could you please sign the EasyCLA?

@nzakas

nzakas commented May 21, 2026

Copy link
Copy Markdown
Member

The CLA is pending because there are commits by more than one entity. I'm guessing the second one is some AI tool?

Please resubmit with a clean commit history.

@snitin315

Copy link
Copy Markdown
Contributor

@monqyClouds Can you please fix the commit history & CLA?

@github-actions

github-actions Bot commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Hi everyone, it looks like we lost track of this pull request. Please review and see what the next steps are. This pull request will auto-close in 7 days without an update.

@github-actions github-actions Bot added the Stale label Jun 6, 2026
@github-actions

Copy link
Copy Markdown
Contributor

This pull request was auto-closed due to inactivity. While we wish we could keep working on every request, we unfortunately don't have the bandwidth to continue here and need to focus on other things. You can resubmit this pull request if you would like to continue working on it.

@github-actions github-actions Bot closed this Jun 13, 2026
@github-project-automation github-project-automation Bot moved this from Triaging to Complete in Triage Jun 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Relates to ESLint's documentation Stale

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

5 participants