Skip to content

Conversation

@rpkilby
Copy link
Contributor

@rpkilby rpkilby commented Nov 29, 2016

No description provided.

Copy link
Collaborator

@carltongibson carltongibson left a comment

Choose a reason for hiding this comment

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

Great. Thanks!

@xordoquy
Copy link
Contributor

I think those cases could have been spotted if the models were in a models.py file.
The went under Django's check radar because of that.
Still need to figure whether it's ok or not.

@carltongibson
Copy link
Collaborator

Still need to figure whether it's ok or not.

I not sure I follow you. 🤔

Docs:

Deprecated since version 1.9: on_delete will become a required argument in Django 2.0. In older versions it defaults to CASCADE.

What's not right? (Or maybe not right?)

@xordoquy
Copy link
Contributor

I though that was required in 1.10.x

@carltongibson
Copy link
Collaborator

But we want it right? (Sorry — I think I need one-more-☕️ this morning...)

@lovelydinosaur
Copy link
Contributor

Great stuff!
Not sure I followed @xordoquy totally there, but we can always follow that up separately.

@lovelydinosaur lovelydinosaur merged commit 6498766 into encode:master Nov 29, 2016
@lovelydinosaur lovelydinosaur added this to the 3.5.4 Release milestone Nov 29, 2016
@rpkilby rpkilby deleted the fix-warnings branch September 25, 2017 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants