Skip to content

Conversation

avivkeller
Copy link
Member

I'm not sure whether we want to enforce exact versioning with our internal dependencies.

This can be approved or rejected as we see fit.

Right now, the dependency versions are locked, as they were very volatile during the web generator creation, however, if we unlock them, it'll allow for less update PRs, but higher chance of breaking changes.

This PR will lock the versions to the exact version for our updating workflow.

@Copilot Copilot AI review requested due to automatic review settings August 6, 2025 00:13
@avivkeller avivkeller requested a review from a team as a code owner August 6, 2025 00:13
Copy link

vercel bot commented Aug 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
api-docs-tooling ✅ Ready (Inspect) Visit Preview Aug 6, 2025 0:13am

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

This PR modifies the internal dependency update workflow to use exact version specifications instead of caret ranges. The change removes the caret (^) prefix from version updates, which will lock dependencies to specific versions rather than allowing compatible updates within the same major version.

  • Removes caret prefix (^) from version specification in the update workflow
  • Changes from compatible version ranges to exact version pinning

Copy link

codecov bot commented Aug 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.27%. Comparing base (82f8545) to head (b48b111).
⚠️ Report is 11 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #394   +/-   ##
=======================================
  Coverage   74.27%   74.27%           
=======================================
  Files         118      118           
  Lines       11041    11041           
  Branches      695      695           
=======================================
  Hits         8201     8201           
  Misses       2837     2837           
  Partials        3        3           

☔ 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.

@avivkeller avivkeller merged commit dc92c22 into main Aug 14, 2025
18 checks passed
@avivkeller avivkeller deleted the avivkeller-patch-1 branch August 14, 2025 13:14
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.

2 participants