The first commit roughly ports the `mkindex.py` to work with Python3 and regenerate the broken `index.json` (it got broken in #57 0c0a8c28bcd3479e472f6125f3362c679cfc895b).
The broken `index.json` caused an error on the theme index page on the Geany website (https://www.geany.org/download/themes/).
The other commits fix older, already existing errors in themes which were not detected because `make index` was not used :(.
In the future, before merging PRs in this repository, the reviewer should take care that `make index` was executed or do it by himself/herself. Also the `AUTHORS` file is not very well maintained.
PR creators and reviewers should carefully read `ADDING-A-THEME.md` before merging. Ideally, there would be a PR template to remind about this. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-themes/pull/58
-- Commit Summary --
* Fix mkindex.py to work with Python3 and regenerate index.json * Fix duplicate key in "retro" theme * Fix screenshot filename for "kary-pro-colors-light" theme
-- File Changes --
M colorschemes/retro.conf (1) M index/index.json (692) M index/index.json.md5 (2) R screenshots/kary-pro-colors-light.png (0) M scripts/mkindex.py (26)
-- Patch Links --
https://github.com/geany/geany-themes/pull/58.patch https://github.com/geany/geany-themes/pull/58.diff
Merged #58 into master.
github-comments@lists.geany.org