[ckan-changes] [ckan/ckan] 555273: 'map' usage made python3 compatible

David Read noreply at github.com
Fri Nov 8 21:32:05 UTC 2019


  Branch: refs/heads/py3-map
  Home:   https://github.com/ckan/ckan
  Commit: 5552739893036faeab10c8ae1c4561ea03011e54
      https://github.com/ckan/ckan/commit/5552739893036faeab10c8ae1c4561ea03011e54
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-11-08 (Fri, 08 Nov 2019)

  Changed paths:
    M ckan/controllers/api.py
    M ckan/include/rjsmin.py
    M ckan/lib/dictization/model_save.py
    M ckan/lib/helpers.py
    M ckan/lib/search/index.py
    M ckan/model/license.py
    M ckan/tests/legacy/functional/test_package.py
    M ckanext/datastore/backend/postgres.py
    M ckanext/example_idatastorebackend/example_sqlite.py

  Log Message:
  -----------
  'map' usage made python3 compatible

Used automated tool:

    python-modernize --fix=map .

As suggested: https://github.com/ckan/ckan/pull/5062#issuecomment-551903816
And did a little manual tidying.


  Commit: 3265c096429e392cf587a7801c3d13be8048eb5f
      https://github.com/ckan/ckan/commit/3265c096429e392cf587a7801c3d13be8048eb5f
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-11-08 (Fri, 08 Nov 2019)

  Changed paths:
    M ckan/lib/dictization/model_dictize.py
    M ckan/tests/legacy/functional/api/model/test_group.py
    M ckan/tests/legacy/functional/api/model/test_vocabulary.py
    M ckan/tests/legacy/lib/test_solr_search_index.py
    M ckan/tests/lib/search/test_index.py
    M ckan/tests/lib/test_datapreview.py
    M ckan/tests/logic/action/test_create.py
    M ckan/tests/logic/action/test_delete.py
    M ckan/tests/logic/action/test_get.py
    M ckan/tests/logic/action/test_update.py
    M ckan/tests/logic/test_conversion.py

  Log Message:
  -----------
  Improvement according to C407 flake8 comprehension rule

C407 Unnecessary list comprehension - 'sorted' can take a generator


Compare: https://github.com/ckan/ckan/compare/fa4b9ceb75ee...3265c096429e


More information about the ckan-changes mailing list