Skip to content

Conversation

adamchainz
Copy link
Member

ticket

Pairs with #12767.

@adamchainz
Copy link
Member Author

@jdufresne you may be interested to check

@carltongibson carltongibson self-assigned this Apr 22, 2020
Copy link
Member

@jdufresne jdufresne left a comment

Choose a reason for hiding this comment

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

Thanks @adamchainz

Copy link
Member

@jdufresne jdufresne Apr 22, 2020

Choose a reason for hiding this comment

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

Should s also be declared? I realize it was not before, so I don't know if it is deliberate or not.

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point, I think it should be.

Copy link
Member

@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.

Yep. Super. Thanks @adamchainz.

@carltongibson carltongibson merged commit 2afa61e into django:master Jun 24, 2020
@adamchainz adamchainz deleted the ticket_31493_docs branch July 20, 2020 10:15
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