Your both util functions are extended to get aditional parameter tab_width, but their signatures different:
def get_set_tab_width_chars(tab_width=None, branch=None):
vs
def get_tab_width_pixels(branch=None, tab_width_chars=None):
As I undersstand in both cases you might pass optional tab_width in chars. I think we should use the same order of arguments in both functions to reduce the possible mistakes in usage of that API. I'm asking you for consistency between those 2 functions. Can you fix this please?
Your both util functions are extended to get aditional parameter tab_width, but their signatures different:
def get_set_ tab_width_ chars(tab_ width=None, branch=None): width_pixels( branch= None, tab_width_ chars=None) :
vs
def get_tab_
As I undersstand in both cases you might pass optional tab_width in chars. I think we should use the same order of arguments in both functions to reduce the possible mistakes in usage of that API. I'm asking you for consistency between those 2 functions. Can you fix this please?