General cleanup #2

Merged
merged 1 commit into from Jan 8, 2012

Conversation

Projects
None yet
2 participants
Contributor

mgrouchy commented Jan 6, 2012

Cleaned up some general stuff picked up by the linter and some other style/unpythonic things I saw.

Still some things that need some work:

  • tasks need to be refactored, there are 3 celery tasks that do essentially the same thing
  • weird things like a million local variables that aren't used
  • Work on making some of the views a little more dry.
  • Move some of the processing from views, to the corresponding forms.

@myusuf3 myusuf3 added a commit that referenced this pull request Jan 8, 2012

@myusuf3 myusuf3 Merge pull request #2 from mgrouchy/master
General cleanup
7548a20

@myusuf3 myusuf3 merged commit 7548a20 into myusuf3:master Jan 8, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment