Docstring for db.models.base.Model.__reduce__() has wrong tense

[jMyles]

Currently reads "Provide pickling support." Compliance with coding guidelines requires "Provides pickling support."

[jMyles]

Looking back, it looks like I did a few things wrong with this ticket - still getting the hang of the social convention here.

*I didn't name the patch .diff
*I marked it fixed, which is supposed to be an action taken by the core team once the patch is rolled in (I have since marked it 're-opened'

Have I done everything else properly?

[Claude Paroz]

While grepping through the code, you'll find that both forms are often used. It might be due to the fact that Django coding guidelines in this case are against PEP 257. I'm not sure it's worth fixing. Or update the guidelines?

[jMyles]

Well, this is an utterly trivial case. I'm not sure what to grep for to determine if non-trivial cases currently exist. There are times when the convention, which is apparently to use the present recurring tense in the indicative mood ("action words") is far more clear than the simple present imperative ("provide spam and eggs to the other classes").

I actually don't really care about the grammar issue all that much; I really just wanted to get my feet wet with opening a ticket and submitting a patch - my hats are, in order: [developer, grammar police]. ;-)

[Adrian Holovaty]

In [16006]:

Fixed #15743 -- Fixed tense in a docstring in db/models/base.py. Thanks, jMyles