Skip to content

docs: remove grep from pull rate limit prerequisites#24455

Open
dvdksn wants to merge 1 commit intomainfrom
fix/issue-24436-remove-grep-prerequisite
Open

docs: remove grep from pull rate limit prerequisites#24455
dvdksn wants to merge 1 commit intomainfrom
fix/issue-24436-remove-grep-prerequisite

Conversation

@dvdksn
Copy link
Contributor

@dvdksn dvdksn commented Mar 21, 2026

Summary

  • Removes grep from the prerequisite list in the "View pull rate and limit" section of content/manuals/docker-hub/usage/pulls.md
  • The three commands shown in that section only use curl and jqgrep is never referenced
  • Fixes a factual error that would cause unnecessary confusion for readers checking whether they have the required tools

Closes #24436

🤖 Generated with Claude Code

@dvdksn dvdksn requested a review from craig-osterhout as a code owner March 21, 2026 10:57
@netlify
Copy link

netlify bot commented Mar 21, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 6e89e9f
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69be79b3e63608000886da8d
😎 Deploy Preview https://deploy-preview-24455--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@dvdksn dvdksn requested a review from a team March 21, 2026 10:57
@github-actions github-actions bot added the area/hub Issue affects Docker Hub label Mar 21, 2026
Copy link
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR correctly removes grep from the prerequisites list. The three commands shown in the "View pull rate and limit" section only use curl and jq:

  1. Getting a token anonymously: curl ... | jq -r .token
  2. Getting a token with authentication: curl --user ... | jq -r .token
  3. Getting headers: curl --head -H "Authorization: Bearer $TOKEN" ...

The grep prerequisite was indeed incorrect and would have confused readers checking whether they have the required tools.

The diff also includes beneficial formatting improvements:

  • Table alignment consistency
  • List indentation fixes
  • Whitespace cleanup

No issues found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[docs-scanner] Incorrect prerequisite list for checking pull limits

1 participant