)]}'
{"validations_libs/cli/base.py":[{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"375522008bcd8aac8e64ff7b873b97181e79d480","unresolved":true,"context_lines":[{"line_number":20,"context_line":"from cliff.lister import Lister"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class BaseCommand(Command):"},{"line_number":24,"context_line":"    \"\"\"Base Command client implementation class\"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def get_parser(self, prog_name):"}],"source_content_type":"text/x-python","patch_set":5,"id":"1c51d0f9_33672087","line":23,"updated":"2021-06-04 10:42:04.000000000","message":"Couldn\u0027t we override this in the existing classes?","commit_id":"29b147e431e101f4de82e01bc2634a08b290bde2"},{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"22885a1ce7ff5a9843c950d7e3a44d2c4b0004be","unresolved":true,"context_lines":[{"line_number":20,"context_line":"from cliff.lister import Lister"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class BaseCommand(Command):"},{"line_number":24,"context_line":"    \"\"\"Base Command client implementation class\"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def get_parser(self, prog_name):"}],"source_content_type":"text/x-python","patch_set":5,"id":"5e20d1f3_d92e662d","line":23,"in_reply_to":"1c51d0f9_33672087","updated":"2021-06-04 11:48:46.000000000","message":"Nop, Command class is used in several places in the CLI, plus, my work with the config file will re-used this base.py file.","commit_id":"29b147e431e101f4de82e01bc2634a08b290bde2"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"b80a1ac57a0fe9ab72d2de9f2a8e8eb1e086062d","unresolved":true,"context_lines":[{"line_number":20,"context_line":"from cliff.lister import Lister"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class BaseCommand(Command):"},{"line_number":24,"context_line":"    \"\"\"Base Command client implementation class\"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def get_parser(self, prog_name):"}],"source_content_type":"text/x-python","patch_set":5,"id":"bf32b441_cee3ca95","line":23,"in_reply_to":"5e20d1f3_d92e662d","updated":"2021-06-04 12:12:26.000000000","message":"I meant in the classes we already define in the validations_libs/cli. They are unique to us and used only in the VF.","commit_id":"29b147e431e101f4de82e01bc2634a08b290bde2"},{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"850f925211b100cd05fb22806c0d501a657fcd4e","unresolved":true,"context_lines":[{"line_number":20,"context_line":"from cliff.lister import Lister"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class BaseCommand(Command):"},{"line_number":24,"context_line":"    \"\"\"Base Command client implementation class\"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def get_parser(self, prog_name):"}],"source_content_type":"text/x-python","patch_set":5,"id":"dab7795e_eef40d57","line":23,"in_reply_to":"bf32b441_cee3ca95","updated":"2021-06-04 12:18:59.000000000","message":"Yep, I\u0027ve understood, but if this piece of code needs to be moved into run.py, it will be duplicated as well in history.py.\nSame for the lister, it will have to be duplicated in several places.\nIt serve actually a common role for all, eventually futur other cli implementations.","commit_id":"29b147e431e101f4de82e01bc2634a08b290bde2"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"b4a4ae708d5e910b93aeaea0a37109b883f89f8f","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from cliff.lister import Lister"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class BaseCommand(Command):"},{"line_number":24,"context_line":"    \"\"\"Base Command client implementation class\"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def get_parser(self, prog_name):"}],"source_content_type":"text/x-python","patch_set":5,"id":"a65f77ef_c649edfb","line":23,"in_reply_to":"dab7795e_eef40d57","updated":"2021-06-04 12:24:53.000000000","message":"Ack","commit_id":"29b147e431e101f4de82e01bc2634a08b290bde2"}]}