Hi Bruno, On 2/26/24 6:38 PM, Bruno Haible wrote: > But now we have the desired configurations of pycodestyle and pylint > in two places: in comments and in the config files. With the promise that > they will get out of sync in the future. To avoid this, let me > update the comments...
True. I thought about adding comments to the configuration files, but your idea is probably better. Less chance for statements to disagree with each other. Better to just tailor the files to those comments. > I don't see a problem here, or anything to "clean up". It is on purpose > that these fields are stored in different objects: > - in the cmdargs, to satisfy the requirements of the argparser, > - in a GLConfig object, for the tool's logic. > For example, the way argparser works, 'mode_list', 'mode_find' etc. > need to be different booleans in cmdargs; however in GLConfig, we don't > want to obey a dictate from argparser; instead it should be a single > field 'mode', with several possible values. I see, it makes more sense with that explanation. Thanks. I saw your email about the sorting of modules and GLImport. I see what you mean and I'll work on that later. It seems that gnulib-tool.py likes to mess with copyright headers on source files copied from lib/. I'll probably look into that first since it makes comparing the result of gnulib-tool and gnulib-tool.py take more work. It seems unrelated to the licensing items in gnulib-tool.py.TODO but those seem trivial so I might as well do them too. Thanks, Collin
