Skip to content
This repository has been archived by the owner on Dec 1, 2021. It is now read-only.

Copyedit the package documentation #135

Merged
merged 1 commit into from
Oct 8, 2018
Merged

Conversation

seh
Copy link
Contributor

@seh seh commented Sep 20, 2017

Remove spurious words, add missing words, and smooth out a few sentences.

I didn't edit any of the function- or type-level documentation in this pass.

Remove spurious words, add missing words, and smooth out a few
sentences.
seh referenced this pull request Sep 20, 2017
@gjnoonan
Copy link

gjnoonan commented Dec 9, 2017

Just trying to help out by cleaning up :-) Can this be merged?

@davecheney davecheney added this to the 0.9 milestone Oct 8, 2018
@davecheney davecheney merged commit 2233dee into pkg:master Oct 8, 2018
@davecheney
Copy link
Member

Thank you very much.

@@ -38,7 +39,7 @@
// }
//
// can be inspected by errors.Cause. errors.Cause will recursively retrieve
// the topmost error which does not implement causer, which is assumed to be
// the topmost error that does not implement causer, which is assumed to be

Choose a reason for hiding this comment

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

Would which be more formal than that when you want to start an object clause?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know about more formal, but it doesn't sound correct to me. In this case, that introduces a defining clause, without which the sentence would possibly make sense but be incorrect: we don't merely retrieve the topmost error; we retrieve the topmost that exhibits this particular characteristic.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants