Skip to content

Conversation

@AdrieanKhisbe
Copy link
Member

Currently, running a bump node to a yarn package cause the introduction of a npm entry.

This is not intended and notably breaks heroku deployments.

This was locally tested, running a bump on a project using yarn.
This should be released as a patch

currently, running a bump node to a yarn package cause the introduction of a npm entry.
This is not intended and notably breaks heroku deployments
@AdrieanKhisbe AdrieanKhisbe requested a review from a team August 5, 2025 09:18
@AdrieanKhisbe AdrieanKhisbe self-assigned this Aug 5, 2025
@AdrieanKhisbe AdrieanKhisbe added Bug 🐛 Not as we expect :/ Enhancement 🍭 Nanofeatures and else ;) labels Aug 5, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.08%. Comparing base (f7f4a37) to head (d17ae33).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #152   +/-   ##
=======================================
  Coverage   75.08%   75.08%           
=======================================
  Files          19       19           
  Lines         618      618           
=======================================
  Hits          464      464           
  Misses        154      154           

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

@AdrieanKhisbe AdrieanKhisbe merged commit f1f0da5 into master Aug 5, 2025
8 checks passed
@AdrieanKhisbe AdrieanKhisbe deleted the fix-node-bump-yarn branch August 5, 2025 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug 🐛 Not as we expect :/ Enhancement 🍭 Nanofeatures and else ;)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants