Skip to content

Copyedit the package documentation#135

Merged
davecheney merged 1 commit intopkg:masterfrom
seh:correct-doc-nits
Oct 8, 2018
Merged

Copyedit the package documentation#135
davecheney merged 1 commit intopkg:masterfrom
seh:correct-doc-nits

Conversation

@seh
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Member

Thank you very much.

//
// 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
Copy link
Copy Markdown

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
Copy Markdown
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 join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants