safe-global / safe-smart-account

Safe allows secure management of blockchain assets.
https://safe.global
GNU Lesser General Public License v3.0
1.84k stars 907 forks source link

safe-singleton-factory should be updated in devDependencies #664

Closed bertux closed 11 months ago

bertux commented 12 months ago

Prerequisites

None because it's documentation fix.

Description

Documentation suggests to run the npm command below to update safe-singleton-factory before deployment but it displays this warning : warning "@gnosis.pm/safe-singleton-factory" is already in "devDependencies". Please remove existing entry first before adding it to "dependencies". I expect documentation to not generate this warning by adding --save-dev option just after npm i. So I have submitted the PR #665 for this missing option.

Environment

Steps to reproduce

npm i @gnosis.pm/safe-singleton-factory

Additional context

None