Skip to content

Conversation

avivkeller
Copy link
Member

Fixes #418 by marking the "About" page as IGNORE_STABILITY_STEMS

@Copilot Copilot AI review requested due to automatic review settings October 12, 2025 14:55
@avivkeller avivkeller requested a review from a team as a code owner October 12, 2025 14:55
Copy link

vercel bot commented Oct 12, 2025

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

Project Deployment Preview Updated (UTC)
api-docs-tooling Ready Ready Preview Oct 12, 2025 2:57pm

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Fixes an issue where definitions on the "About" page were incorrectly being marked as deprecated by excluding specific file stems from stability index processing.

  • Added a constant to define file stems that should ignore stability processing
  • Modified stability metadata parsing to conditionally skip processing based on file stem
  • Updated function signature to handle optional metadata parameter

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/generators/metadata/constants.mjs New file defining stems to ignore for stability processing
src/generators/metadata/utils/parse.mjs Added logic to conditionally skip stability processing for specific files
src/utils/queries/index.mjs Updated function to handle optional metadata parameter with null safety

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

codecov bot commented Oct 12, 2025

Codecov Report

❌ Patch coverage is 54.54545% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.53%. Comparing base (3291812) to head (b99b33a).

Files with missing lines Patch % Lines
src/generators/metadata/utils/parse.mjs 16.66% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #456      +/-   ##
==========================================
- Coverage   74.55%   74.53%   -0.02%     
==========================================
  Files         107      108       +1     
  Lines       10466    10474       +8     
  Branches      684      685       +1     
==========================================
+ Hits         7803     7807       +4     
- Misses       2661     2665       +4     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

stability index is mark as deprecated

3 participants