Skip to content

Conversation

@tavva
Copy link

@tavva tavva commented Mar 26, 2014

No description provided.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using a previous django version isn't exactly an exceptional situation, so rather than handling an exception here you ought to be using getattr.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also missing "pass" or something so this isn't a hanging block

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pass is only used to mark an empty block as being empty. This isn't an empty block.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh it got interrupted by these comments

@giftig
Copy link

giftig commented Apr 2, 2014

Please indicate what has changed in the commit message

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants