Skip to content

chore: Fix punctuation and phrasing in messages

Amy Qualls requested to merge 7470-aqualls-post-merge-review into main

Description

I … uh … oh dear. This MR will NOT bring delight to @garyh's face, I fear. Tomas reminded me in !1517 (comment 1951010899) to do a post-merge review of the docs and string changes.

I ran grep -riI "fmt.Errorf(" . thinking, "oh, these strings will be fine." Reader, it was not fine. I found a lot of punctuation fixes. This MR is the first half; even I realized I should stop before committing more changes.

Let's find out how many tests I broke…

Related Issues

Related to feat(stacked-diffs): Create initial sync command (!1517 - merged) where Tomas reminded me to do a post-merge review on the strings.

How has this been tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)
  • Test gap

Closes #7470 (closed)

Edited by Amy Qualls

Merge request reports