Skip to content
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: bin path for fuels-forc in fuels CLI, add support bun's structure #1481

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

Dhaiwat10
Copy link
Contributor

Similar to #1477, but also adds (possible) support for bun, which has a different way of structuring binaries inside of node_modules.

@Dhaiwat10 Dhaiwat10 enabled auto-merge (squash) December 6, 2023 00:54
@Dhaiwat10 Dhaiwat10 self-assigned this Dec 6, 2023
@Dhaiwat10 Dhaiwat10 added the bug Issue is a bug label Dec 6, 2023
Copy link
Contributor

github-actions bot commented Dec 6, 2023

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
86.41% (-0.02% 🔻)
5838/6756
🟡 Branches
69.79% (-0.11% 🔻)
864/1238
🟡 Functions 77.12% 964/1250
🟢 Lines
86.38% (-0.02% 🔻)
5588/6469
Show new covered files 🐣
St.
File Statements Branches Functions Lines
🟢
... / findBinPath.ts
87.5% 0% 100% 87.5%

Test suite run success

1398 tests passing in 252 suites.

Report generated by 🧪jest coverage report action from c150a21

@Dhaiwat10 Dhaiwat10 merged commit 014c27f into master Dec 6, 2023
@Dhaiwat10 Dhaiwat10 deleted the fix/bin-paths branch December 6, 2023 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants