Skip to content

Conversation

@osv-robot
Copy link
Collaborator

The snapshots and cassettes have changed, probably due to OSV advisories being changed.
Please review the differences to make sure that they're expected!

@codecov-commenter
Copy link

codecov-commenter commented Dec 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.78%. Comparing base (c82c9c9) to head (b0184aa).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2384      +/-   ##
==========================================
+ Coverage   67.75%   67.78%   +0.03%     
==========================================
  Files         172      172              
  Lines       13291    13291              
==========================================
+ Hits         9005     9009       +4     
+ Misses       3581     3579       -2     
+ Partials      705      703       -2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@osv-robot osv-robot force-pushed the bot/update-snapshots branch 3 times, most recently from 5217e73 to aa85ff7 Compare December 4, 2025 19:03
@G-Rath
Copy link
Collaborator

G-Rath commented Dec 4, 2025

@another-rex I'm wondering if we split out the "update snapshots" and "update cassettes" jobs, since it is still possible for the snapshots to change without the cassettes changing, and those should now be a smaller diff so easier to land 🤔

@another-rex
Copy link
Collaborator

Yeah good idea, can you create a new workflow?

I'm thinking cassette + snapshots, and just snapshots.

@osv-robot osv-robot force-pushed the bot/update-snapshots branch from aa85ff7 to b0184aa Compare December 5, 2025 19:00
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.

4 participants