fireprox icon indicating copy to clipboard operation
fireprox copied to clipboard

Refactor to make more import friendly

Open chm0dx opened this issue 2 years ago • 0 comments

Some minor changes to make things more import friendly. The experience should be nearly identical when invoked from the command line but improved when imported.

  • Prints relegated to main() with results returned from associated functions rather than printed directly
  • FireProx.error() raises a custom FireProxException, handled and printed with try/except in main()
  • Requirement for a command argument restricted to main()
  • FireProx constructor no longer requires an argparse.Namespace argument so that FireProx() can be called directly without jumping through hoops

Thanks!

chm0dx avatar Nov 30 '22 03:11 chm0dx