0LNetworkCommunity / libra-legacy-v6

Apache License 2.0
248 stars 116 forks source link

[Chore] Refactor `.unwrap()` with Match in `/ol/util` folder #1193

Open 0xzoz opened 1 year ago

0xzoz commented 1 year ago

Bounty - 2000 GAS

What is the issue

During the development of the initial implementation of 0L. Work was done quickly to meet deadlines and because of this .unwrap() was used in some instances to speed up development. These need to be cleaned up.

What needs to be done?

Intention

This issue is part of the Tool Scrubbers Guild

The idea behind this issue is to create manageable first tasks for relatively novice prospective developers looking to learn more about Rust and 0L. Not all of the unwraps will need to be refactored and part of the exercise is determining if the .unwrap() could be refactored or if they are already best practice depending on the situation

0xzoz commented 1 year ago

@mortonbits this is where the subfolder from #1144 has been moved. Could you please confirm if you are still doing this task, completed(tag PR if you have) or if are no longer going to complete it. Thanks

shawnharmsen commented 1 year ago

I'd love to take a stab at this, will add a PR shortly.