Skip to content

Seperate out args parser code from main.rs #9

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Merged
merged 2 commits into from
Jun 4, 2023
Merged

Seperate out args parser code from main.rs #9

merged 2 commits into from
Jun 4, 2023

Conversation

nozwock
Copy link
Contributor

@nozwock nozwock commented Jun 3, 2023

Based off #8

@nozwock
Copy link
Contributor Author

nozwock commented Jun 3, 2023

Let me know if I should do any further refactors.

@dvishal485
Copy link
Owner

Thanks for your contribution! This looks good to me.
And respect for reducing a level of indentation and per file size. 🙇🏻

@dvishal485 dvishal485 merged commit b324bf8 into dvishal485:main Jun 4, 2023
@nozwock nozwock deleted the refactor branch June 4, 2023 08:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nozwock @dvishal485