mitchmindtree

Results 396 comments of mitchmindtree

Could you clarify what "configuration time" means and how it differs from compile-time? IIRC, this is about being able to more easily change between certain constants like addresses or IDs...

Rather than writing directly to `stdout` (as is the case in `fuelup`), we should instead see if we can write formatted output via `tracing` to allow for formatted output to...

I think we can leave this one open as this issue is more about taking on `fuelup`'s approach to handling coloured and bold writing. See [this fuelup module](https://github.com/FuelLabs/fuelup/blob/57021980d5917c8f0507860e90d80dc2dc0ae2f3/src/fmt.rs) for an...

Ahh thanks for digging into this @sdankel and clarifying! Good point about `tracing` not exposing any kind of stream abstraction - happy to close this in favour of your suggested...

Cargo also provides a `--color` flag for controlling this, we might want to consider doing something similar too. We should consider addressing #2373 first as it would simplify adding an...

Originally these subcommands were introduced to `cargo` via an external plugin called `cargo-edit` - perhaps we can take the same approach. That said, it looks like `add` has since been...

I think the intermediate implementations are close enough but could just use a little tweaking for readability - see [this comment](https://github.com/FuelLabs/sway/pull/2440#discussion_r943074559).

> > Create a new sway-test-rs repo or similar that can be used as a cargo generate template. > > Should this be a directory in fuels-rs proper? I think...

Just an update for reviewers: Putting this on hold until we land wallet integration in forc deployment. Will come back to this PR after addressing https://github.com/FuelLabs/sway/issues/2412 and https://github.com/FuelLabs/sway/issues/2413.

OK! Just transferred the `sway-test-rs` cargo generate template repository to FuelLabs, you can find it here: https://github.com/fuellabs/sway-test-rs. I've added some usage steps to its README, just need to update the...