…and add a script to automate the process. This script could be made a little cleaner, but it works just fine (and actually didn't alter anything if only including Python 2 keywords).
Part of #748. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/755
-- Commit Summary --
* Add a script to update Python filetype's identifiers list * python: Update identifiers for Python 3
-- File Changes --
M data/filetypes.python (2) A scripts/update-python-identifiers.sh (26)
-- Patch Links --
https://github.com/geany/geany/pull/755.patch https://github.com/geany/geany/pull/755.diff
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/755
See https://github.com/geany/geany/pull/748#issuecomment-156162645, as maybe listing everything here as we used to is not the best way to go forward -- or maybe it is.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/755#issuecomment-156163226
LGTM, especially the script for updating is nice!
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/755#issuecomment-156247143
LGTM, except the PY3 update commit leaves the comment on the line above saying py 2.7 keywords.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/755#issuecomment-156250245
@elextr good catch, fixed
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/755#issuecomment-156748607
…and while at it, let's update keywords as well as we can query those too.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/755#issuecomment-156756302
(which adds missing `nonlocal` keyword)
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/755#issuecomment-156756335
Merged #755.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/755#event-464644292
github-comments@lists.geany.org