search

flux-framework / flux-accounting

bank/accounting interface for the Flux resource manager
https://flux-framework.readthedocs.io/projects/flux-accounting/en/latest/index.html
GNU Lesser General Public License v3.0
3 stars 10 forks source link

`list-banks`: use `AccountingFormatter` class #524

Closed cmoussa1 closed 1 week ago

cmoussa1 commented 3 weeks ago

Problem

The list-banks command defines its own formatting for the output of the command, but the Python bindings now has its own common formatter and SQL utilities.

This PR edits the list-banks command to make use of the new formatter and external SQLite utilities. It also changes the default for the --fields optional argument to be None if it is not specified.

cmoussa1 commented 1 week ago

Thanks @wihobbs!! Rebased to catch up. Setting MWP here