Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Chore] Remove/refactor .clone() from the /ol/tower folder where needed #1202

Open
0xzoz opened this issue Nov 5, 2022 · 0 comments
Open

Comments

@0xzoz
Copy link
Collaborator

0xzoz commented Nov 5, 2022

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 .clone() was used in some instances to speed up development. These need to be cleaned up.

What needs to be done?

  • Go through the /ol/tower folder in root and remove instances of .clone() by refactoring the code. This may include refactoring of related files that the code is coupled to.

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 .clone() could be refactored or if they are already best practice depending on the situation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant