Skip to content
This repository has been archived by the owner on Jul 19, 2021. It is now read-only.

replace import * with named imports where feasible #1

Merged
merged 1 commit into from Sep 20, 2016
Merged

replace import * with named imports where feasible #1

merged 1 commit into from Sep 20, 2016

Conversation

ghost
Copy link

@ghost ghost commented Sep 20, 2016

There are way too many GeoNode settings and too much change in them to make it
convenient to import them all explicitly, which is why that instance in _prj is
left behind (but shouldn't matter a lot anyway because of what _prj is for).

flake8 still needs # noqa because of F401 about things being unused, those
are impossible to avoid without sacrificing the stated interest of using
geonode_apis to expose versions of things which may be adjusted for GeoNode

There are way too many GeoNode settings and too much change in them to make it
convenient to import them all explicitly, which is why that instance in _prj is
left behind (but shouldn't matter a lot anyway because of what _prj is for).

flake8 still needs # noqa because of F401 about things being unused, those
are impossible to avoid without sacrificing the stated interest of using
geonode_apis to expose versions of things which may be adjusted for GeoNode
@sarasafavi sarasafavi merged commit 866911b into planetfederal:master Sep 20, 2016
@ghost ghost deleted the replace_import_star branch September 23, 2016 00:15
@ghost ghost mentioned this pull request Sep 23, 2016
ghost pushed a commit that referenced this pull request Oct 31, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant