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

Small refactoring #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Small refactoring #1

wants to merge 1 commit into from

Conversation

JH2535
Copy link

@JH2535 JH2535 commented Mar 3, 2022

Refactored code, pulling out some methods to make slightly more readable / reduce file size.

Start at bottom method (jig), work your way up when needed.

Also provided a default example (from the video) for anyone who is just pulling down the code and running it (as currently all the code does was load methods into memory).

Feedback and requests are welcome! Sorry for the messy commit, typically I would break it down into smaller chunks and send them bit by bit but I was a bit rushed for time. For a better viewing on the commits tab - click on the commit, then the tripple dots and click view file, it will give a much better idea as to what the updated file will look like.

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

Successfully merging this pull request may close these issues.

None yet

1 participant