Skip to content
/ django Public

Address #16855 - Deprecate depth kwarg on select_related.#488

Closed
mjtamlyn wants to merge 1 commit intodjango:masterfrom
mjtamlyn:deprecate-depth
Closed

Address #16855 - Deprecate depth kwarg on select_related.#488
mjtamlyn wants to merge 1 commit intodjango:masterfrom
mjtamlyn:deprecate-depth

Conversation

@mjtamlyn
Copy link
Copy Markdown
Member

@mjtamlyn mjtamlyn commented Nov 2, 2012

This is the start of a deprecation path for the depth kwarg on select_related. Removing this will allow us to update select_related so it chains properly and have a more similar API to prefetch_related.

This is the start of a deprecation path for the depth kwarg on
select_related. Removing this will allow us to update select_related so
it chains properly and have a more similar API to prefetch_related.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should be a PendingDeprecationWarning at this time. (Also a \n would help readability.)

@ptone
Copy link
Copy Markdown
Member

ptone commented Nov 3, 2012

merged with some extra docs modifications in 965cc0b

@ptone ptone closed this Nov 3, 2012
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