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.

Sign up for GitHub

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

Jump to bottom

refactor: arg parsing & add integration tests #102

Merged
merged 16 commits into from
Dec 30, 2021
Merged

refactor: arg parsing & add integration tests #102

merged 16 commits into from
Dec 30, 2021

Conversation

disrupted
Copy link
Owner

  • chore: move test resources
  • chore: add test sub
  • refactor: extract arg parser to function
  • refactor: cli
  • refactor: let argparse handle sys.argv
  • refactor: accept args in main method
  • chore: rename sub
  • refactor: pass file as path instead of str
  • test: create integration test
  • chore: add another example
  • refactor: run style fixes in separate processor at the end
  • fix: update reference sub

@disrupted disrupted added the enhancement New feature or request label Dec 30, 2021
@disrupted disrupted self-assigned this Dec 30, 2021
@disrupted disrupted changed the title test/integration test: integration Dec 30, 2021
@disrupted disrupted changed the title test: integration refactor: arg parsing & add integration tests Dec 30, 2021
@disrupted disrupted merged commit 0b0ea27 into main Dec 30, 2021
@disrupted disrupted deleted the test/integration branch December 30, 2021 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@disrupted disrupted

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@disrupted