Skip to content

Refactor CmdArgs.run()#196

Merged
VerisimilitudeX merged 5 commits intoVerisimilitudeX:mainfrom
ishche:main
Oct 17, 2022
Merged

Refactor CmdArgs.run()#196
VerisimilitudeX merged 5 commits intoVerisimilitudeX:mainfrom
ishche:main

Conversation

@ishche
Copy link
Contributor

@ishche ishche commented Oct 12, 2022

Closes #184

@CLAassistant
Copy link

CLAassistant commented Oct 12, 2022

CLA assistant check
All committers have signed the CLA.

@ishche ishche closed this Oct 12, 2022
@VerisimilitudeX
Copy link
Owner

@ishche why did you close this PR? Is there anything I can do to help?

@ishche
Copy link
Contributor Author

ishche commented Oct 13, 2022

@Verisimilitude11 oh, PR checks reported that refactoring is still needed.

@VerisimilitudeX
Copy link
Owner

Don't worry about that, the checks aren't that accurate.

@VerisimilitudeX
Copy link
Owner

I'll merge it after some testing later today.

@VerisimilitudeX
Copy link
Owner

Could you please clear the conflicts? Thanks!

@VerisimilitudeX
Copy link
Owner

Hey @ishche, I will be out of office for some time, so it will take me some time merge the pull request. However, I will approve the pull request in case you are participating in Hacktoberfest. Thanks for working on this, we at DNAnalyzer really appreciate your help!

@VerisimilitudeX
Copy link
Owner

@ishche could you please star this repository of you haven’t already.

@VerisimilitudeX VerisimilitudeX merged commit d7f226f into VerisimilitudeX:main Oct 17, 2022
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.

Refactor CmdArgs.run()

3 participants