Skip to content
This repository has been archived by the owner on Nov 5, 2019. It is now read-only.

Import clean up #625

Merged
merged 2 commits into from
Aug 11, 2016
Merged

Import clean up #625

merged 2 commits into from
Aug 11, 2016

Conversation

pferate
Copy link
Contributor

@pferate pferate commented Aug 11, 2016

  • Treat tests as part of the local package (for import order)
  • Import modules instead of objects (some slipped by since this was cleaned up previously)

Some imports that slipped by last time this was cleaned up.
@theacodes
Copy link
Contributor

lgtm. I don't know why django-related code always looks so gross to me.

@theacodes
Copy link
Contributor

(waiting for the test god to merge)

@pferate
Copy link
Contributor Author

pferate commented Aug 11, 2016

Flask > Django?

@theacodes
Copy link
Contributor

Flask > Django?

At least to me. I've never been much of a fan of Django (though I love their community!)

@theacodes theacodes merged commit c6b30bf into googleapis:master Aug 11, 2016
@pferate pferate deleted the import_formatting branch August 11, 2016 21:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants