Skip to content

Cache PyPI shields for 12 hours - #24

Open
rubytobi wants to merge 1 commit into
sqlfluff:mainfrom
rubytobi:cached-shields
Open

rubytobi wants to merge 1 commit into
sqlfluff:mainfrom
rubytobi:cached-shields

Conversation

@rubytobi

@rubytobi rubytobi commented Sep 8, 2026

Copy link
Copy Markdown

I'm always greeted with a rate limited by upstream service shield looking at the index page.

Once can actually configure a cacheSeconds via URL parameters.

HTTP cache lifetime (rules are applied to infer a default value on a per-badge basis, any values specified below the default will be ignored).
https://shields.io/badges/dynamic-json-badge

image

This change sets the lifetime to 12 hours for both PyPI downloads and version shields.

You can't see the improvement yet until the rate limit is over. It should prevent future rate limits from recurring.

@netlify

netlify Bot commented Sep 8, 2026

Copy link
Copy Markdown

Deploy Preview for sqlfluff ready!

Name Link
🔨 Latest commit 00232f4
🔍 Latest deploy log https://app.netlify.com/projects/sqlfluff/deploys/6a9f67d9de2f660008ab88f4
😎 Deploy Preview https://deploy-preview-24--sqlfluff.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.

@rubytobi
rubytobi marked this pull request as ready for review September 8, 2026 01:46

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

Re-trigger cubic

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.

1 participant