search

FuelLabs / fuels-ts

Fuel Network Typescript SDK
https://docs.fuel.network/docs/fuels-ts/
Apache License 2.0
44.4k stars 1.32k forks source link

chore!: upgrade `commander` to`12.1.0` #2620

Closed YaTut1901 closed 6 days ago

YaTut1901 commented 1 week ago

Breaking Changes:

The shorthand flag (-p) for passing through a path to various fuels CLI commands has been deprecated. Migrate to using the --path flag instead.

maschad commented 1 week ago

Thanks for opening this @YaTut1901 , I've converted this to a draft PR. Feel free to re-open once the CI tests are passing.

YaTut1901 commented 6 days ago

Worth making the breaking change description a little bit more descriptive:

The shorthand flag (-p) for passing through a path to various fuels CLI commands has been deprecated. Migrate to using the --path flag instead.

do you mean description in PR?

petertonysmith94 commented 6 days ago

Worth making the breaking change description a little bit more descriptive:

The shorthand flag (-p) for passing through a path to various fuels CLI commands has been deprecated. Migrate to using the --path flag instead.

do you mean description in PR?

Correct 🔥

petertonysmith94 commented 6 days ago

Failing due to the following check: Release to @rc-<pr_number> tag on npm / Release breaking change PR to npm (pull_request) (source).

Will merge anyway as I don't see any breaking change for the wallet here.