Update documentation that news entries cannot be empty files

[QuentinLee5]

Currently, the documentation regarding news entries for trivial PRs mentions that contributors can create an empty news entry using for example the following command on Linux touch news/$(uuidgen).trivial.rst.

The check for a news entry in PRs however does not longer allow empty .trivial files.

https://github.com/pypa/pip/blob/master/docs/html/development/contributing.rst needs to be updated according to this.

[pfmoore]

I'd actually say that the correct fix is to allow empty .trivial files again. I recall there was a discussion about this somewhere, and maybe there's a reason we aren't going to do that, but can a link to the reason be added here, if that's the case?

[pradyunsg]

Well, this is a good stopgap solution until we fix the issue on the bot. 🤷🏻‍♂️

[pfmoore]

I'm fine with that. I just wasn't sure if it was a temporary thing.

[pradyunsg]

I'm definitely viewing this as temporary, but it also won't hurt to just have people follow this long term. :)

[QuentinLee5]

So we should change the documentation that contributors should add a news entry at news/$(uuidgen).trivial.rst with a small description explaining what the PR changes (or would a non-empty news entry be sufficient)?

[pfmoore]

Let's be explicit here, and just say that a .trivial must contain something, but that's due to a bug in the validation, and the information in there is ignored, so a blank space is sufficient.

[ichard26]

Well, this is a good stopgap solution until we fix the issue on the bot. 🤷🏻‍♂️

ref: pypa/browntruck#24

[wanghaojiang1]

This issue seems like a good first issue for beginning contributers. I would like to work on this issue.

[ichard26]

It appears that the new changelog GitHub App check handles empty files just fine: https://github.com/pypa/pip/pull/12534/files#diff-8a3182a318bebebaebb86d2efe42c6594eb98182b87d1a99e66a34b61705ed33.

[uranusjr]

Yes, this has been fixed a while ago. Thanks for finding this!