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 -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-themes/pull/58 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany-themes/pull/5...@github.com>