Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

gps: improve documentation; add package doc #1619

Merged
merged 1 commit into from
Feb 8, 2018
Merged

gps: improve documentation; add package doc #1619

merged 1 commit into from
Feb 8, 2018

Conversation

jmank88
Copy link
Collaborator

@jmank88 jmank88 commented Feb 2, 2018

What does this do / why do we need it?

This PR improves and adds package gps documentation, based on scanning through the godoc.

What should your reviewer look out for in this PR?

  • More/better info to add.
  • Were any of the sentences I punctuated cut off?

Do you need help or clarification on anything?

Should links to https://github.com/sdboyer/gps/wiki be forwarded elsewhere? Or is that still the best resource?

Which issue(s) does this PR fix?

Towards #672.

// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

// Package gps is a Go packaging solver library.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Embedding code examples here could get pretty dense, but maybe this should point to example.go at least (especially since it doesn't show up in the go doc as a runnable example).

@jmank88 jmank88 closed this Feb 2, 2018
@jmank88 jmank88 reopened this Feb 2, 2018
Copy link
Collaborator

@darkowlzz darkowlzz left a comment

Choose a reason for hiding this comment

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

LGTM!

@jmank88 jmank88 merged commit 93bcb36 into golang:master Feb 8, 2018
@jmank88 jmank88 deleted the gps_doc branch February 8, 2018 13:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants