Skip to content

fix(nx-dev): handle null github data#33061

Merged
barbados-clemens merged 2 commits intomasterfrom
cu/DOC-241-gh-stars
Oct 14, 2025
Merged

fix(nx-dev): handle null github data#33061
barbados-clemens merged 2 commits intomasterfrom
cu/DOC-241-gh-stars

Conversation

@barbados-clemens
Copy link
Copy Markdown
Contributor

@barbados-clemens barbados-clemens commented Oct 13, 2025

technically the fix to plugin stats now showing up was expired GH token.
But refactored the plugin stats fetching to skip locally unless NX_DOCS_PLUGIN_STATS env var is set to help speed up local serves/builds since 99% of the time we're not concerned with the plugin-registry page.
along with trying to centralized the logic between 1st/3rd party plugins since it was a little confusing from my initial impl.

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Oct 14, 2025 1:40pm

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 13, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit e46c6d7
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/68ee4f2e8343720008777bc4
😎 Deploy Preview https://deploy-preview-33061--nx-docs.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.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Oct 13, 2025

View your CI Pipeline Execution ↗ for commit e46c6d7

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 2m 56s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 2m 16s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx documentation ✅ Succeeded 20s View ↗

☁️ Nx Cloud last updated this comment at 2025-10-14 13:34:13 UTC

@barbados-clemens barbados-clemens merged commit 61b2796 into master Oct 14, 2025
12 checks passed
@barbados-clemens barbados-clemens deleted the cu/DOC-241-gh-stars branch October 14, 2025 14:46
FrozenPandaz pushed a commit that referenced this pull request Oct 15, 2025
technically the fix to plugin stats now showing up was expired GH token.
But refactored the plugin stats fetching to skip locally unless
NX_DOCS_PLUGIN_STATS env var is set to help speed up local serves/builds
since 99% of the time we're not concerned with the plugin-registry page.
along with trying to centralized the logic between 1st/3rd party plugins
since it was a little confusing from my initial impl.

(cherry picked from commit 61b2796)
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants