-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: improve fuels init
directory detection
#3638
fix: improve fuels init
directory detection
#3638
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
0d8a4d5
to
af123ca
Compare
@petertonysmith94 Maybe I am missing something, but when running this locally while using the ➜ pnpm fuels init --predicates sway-programs/predicate/* --output ./src/predicate_api
using deprecated parameters for the initialization function; pass a single object instead
✅ Your fuels version is up to date: 0.98.0
############### INIT CALLED
error: unable to detect program/s
- contract/s detected 0 The But when using the released version, everything seems to work: ➜ pnpm fuels init --predicates sway-programs/predicate/* --output ./src/predicate_api
using deprecated parameters for the initialization function; pass a single object instead
✅ Your fuels version is up to date: 0.98.0
Config file created at:
/Users/torres-m3/Documents/personal/my-fuel-project/fuels.config.ts |
It was me that was missing something - nice catch 🎣 Turns out the OS expands glob patterns before they hit the CLI. Therefore, our tests weren't picking these up. I've made the following changes 167c42a which should address this :) |
Coverage Report:
Changed Files:
|
fuels init
directory detection #3614Release notes
In this release, we:
fuels init
Summary
When using
fuels init
, we now detect individualcontracts
,predicates
andscripts
by their respectiveForc.toml
files. This should account for more user error when using the command and alleviate some pain points.Checklist