pgi-docgen icon indicating copy to clipboard operation
pgi-docgen copied to clipboard

Fix mergeindex list issue

Open fcole90 opened this issue 2 years ago • 0 comments

Hi and thanks for this awesome library.

While running it, I encountered the following error:

$ ./tools/build.sh Gtk-3.0
All done
Traceback (most recent call last):
  File "/home/fabio/Projects/scratches/pgi-docgen/./pgi-docgen", line 14, in <module>
    sys.exit(main(sys.argv))
  File "/home/fabio/Projects/scratches/pgi-docgen/pgidocgen/main.py", line 29, in main
    return args.func(args)
  File "/home/fabio/Projects/scratches/pgi-docgen/pgidocgen/build.py", line 260, in main
    mergeindex(target_path)
  File "/home/fabio/Projects/scratches/pgi-docgen/pgidocgen/mergeindex.py", line 170, in mergeindex
    output = merger.merge()
  File "/home/fabio/Projects/scratches/pgi-docgen/pgidocgen/mergeindex.py", line 123, in merge
    for attr, v in attributes.items():
AttributeError: 'list' object has no attribute 'items'

It seems the mergeindex file was mistaking a list for a dictionary. I added some typing to aid me with fixing the issue. I think this mostly makes sense (and the error is no longer there) but I'm not 100% sure about the output type. I can say that it's now coherent with the input and at least produces a final merged index :blush:

fcole90 avatar Jul 10 '22 15:07 fcole90