You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In your opinion, how should changelist handle cases were the PR summary consists of more than one sentence? Delimit the leading sentences with ". " but omit from the last one?
Feels a bit too inconsistent and grammatically wrong to me, to be worth saving on visual noise.. my personal impression obviously. ;)
I feel like we discussed this before. I think I was advocating for a style similar to NumPy:
Some summary. Possibly with multiple sentences. ([#PR number](link))
For me the closing "." encourages to view of release notes as more than a list of scraped PR titles and as stated feels more consistent and correct. We are also doing it that way everywhere else in our documentation, e.g. parameter descriptions.
If you want to omit full-stops for the "visual noise" reason I'm -0.1, but if you want to have the option to list the summary as is without modification, than I am happy to add it as a configuration option. What do you think?
Unsolicited opinion from the sideline, but I also like full sentence(es) with a closing full stop before the PR number.
But maybe it's just the style I'm used to.
- Really short summary, preferably a single sentence. [[# PR number](link)]
Adds a bit of visual noise, when the hashtag-PR-number at the end of the line already indicates a clear line termination
The text was updated successfully, but these errors were encountered: