-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: use env BUN_INSTALL to locate package.json #734
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #734 +/- ##
========================================
- Coverage 5.48% 5.48% -0.01%
========================================
Files 37 37
Lines 11860 11862 +2
========================================
Hits 651 651
- Misses 11209 11211 +2 ☔ View full report in Codecov by Sentry. |
cc @jtrv, here is build with this patch included, welcome to give it a test: https://github.com/SteveLauC/topgrade/releases/tag/PR734 |
@SteveLauC sorry I just saw this. This PR does not solve the issue for me. However I just wrote my own PR which works correctly in my testing: #735 |
Would you like to run this binary: #734 (comment) with: $ ./topgrade --only bun_packages --verbose and give me the output? |
|
So, that log does not help too much, what is the path of your And what is the value of your $ echo $BUN_HOME |
In your PR, I see you are using shellexpand, does bun recognize shell signs like |
My apologies @SteveLauC I realized I compiled on the wrong branch, this works and it looks better than my PR, I'll close mine 👍🏽 |
Standards checklist:
CONTRIBUTING.md
cargo build
)cargo fmt
)cargo clippy
)cargo test
)For new steps
--dry-run
option works with this step--yes
option works with this step if it is supported bythe underlying command
If you developed a feature or a bug fix for someone else and you do not have the
means to test it, please tag this person here.
Closes #733