Skip to content

Conversation

@runningcode
Copy link
Contributor

@runningcode runningcode commented Jan 30, 2026

Summary

Adds documentation for configuring status check rules in the Size Analysis CI integration guide. This feature allows users to define thresholds that automatically fail status checks when exceeded on pull requests.

Documentation Updates

  • CI Integration Guide (integrating-into-ci.mdx):
    • New "Configuring Status Check Rules" section
    • Documents how to access status check settings
    • Explains rule components: Metrics, Measurement Types, Thresholds, and Filters
    • Provides practical example (preventing 5 MB increases)
    • Explains multiple rules behavior and default behavior

Add comprehensive documentation for configuring status check rules in the Size Analysis CI integration guide. This allows users to set up automatic status check failures when size thresholds are exceeded.

Changes:
- Add "Configuring Status Check Rules" section to CI integration guide
- Document metrics (Install Size, Download Size)
- Document measurement types (Absolute Size, Absolute Diff, Relative Diff)
- Document optional filters (App ID, Branch, Build Configuration, Platform)
- Include practical example of preventing large size increases
- Update index.mdx to mention configurable rules
- Upgrade platformicons to 9.0.6 to fix missing litellm.svg icon
- Add screenshots for settings page and rule configuration form

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Jan 30, 2026

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

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment Jan 30, 2026 4:28pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Preview Jan 30, 2026 4:28pm

Request Review

"nextjs-toploader": "^1.6.6",
"p-limit": "^6.2.0",
"platformicons": "^9.0.5",
"platformicons": "^9.0.6",
Copy link
Contributor Author

@runningcode runningcode Jan 30, 2026

Choose a reason for hiding this comment

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

I was having a compile issue and claude suggested this fix and then it compiled 🤷

@runningcode runningcode marked this pull request as ready for review January 30, 2026 16:05
@runningcode runningcode requested review from chromy and mtopo27 January 30, 2026 16:06
Copy link
Contributor

@chromy chromy left a comment

Choose a reason for hiding this comment

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

lgtm

Replace `//` with proper YAML comment syntax `#` in GitHub Actions workflow examples. The previous syntax would cause YAML parsing errors if copied directly.

Co-Authored-By: Claude <noreply@anthropic.com>
@runningcode runningcode merged commit 5cdac3c into master Jan 30, 2026
14 checks passed
@runningcode runningcode deleted the no/document-status-check-rules branch January 30, 2026 16:30
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.

3 participants