Unifying function to get Safe implementation

safe-global / safe-smart-account

Safe allows secure management of blockchain assets.

https://safe.global

GNU Lesser General Public License v3.0

1.88k stars 927 forks source link

Unifying function to get Safe implementation #808

Closed remedcu closed 3 months ago

remedcu commented 3 months ago

Fixes #802

This PR unifies getSafeWithOwners(...) and getSafeWithSingleton(...) by editing the getSafeTemplate(...) to use an inside function which also takes the singleton as a parameter.

coveralls commented 3 months ago

Pull Request Test Coverage Report for Build 10298933635

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.478%

Totals
Change from base Build 10196375209:	0.0%
Covered Lines:	386
Relevant Lines:	398

safe-global / safe-smart-account

Unifying function to get Safe implementation #808

Pull Request Test Coverage Report for Build 10298933635

Details

💛 - Coveralls