Skip to content

Improve consistency of "GitHub API docs" strings in comments. #562

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 23, 2017

Conversation

dmitshur
Copy link
Member

@dmitshur dmitshur commented Feb 23, 2017

While reviewing #551, I noticed that there were various slight variations in the phrasing of comments with the form:

// GitHub API docs: https://developer.github.com/v3/...

This change makes them all consistent.

Also fix "GitHub" misspelled as "Github" in a few places.

Followup to #551. /cc @kevinburke

There were various slight variations in the phrasing of comments with
the form:

	// GitHub API docs: https://developer.github.com/v3/...

This change makes them all consistent.

Also fix "GitHub" misspelled as "Github" in a few places.

Followup to #551.
Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Very nice! Thank you, @shurcooL!

@gmlewis gmlewis added the lgtm label Feb 23, 2017
@dmitshur dmitshur self-assigned this Feb 23, 2017
@dmitshur dmitshur merged commit b59d3a6 into master Feb 23, 2017
@dmitshur dmitshur deleted the consistent-GitHub-API-docs branch February 23, 2017 16:22
@dmitshur dmitshur removed their assignment Feb 23, 2017
varadarajana added a commit to varadarajana/go-github that referenced this pull request Mar 4, 2017
gmlewis pushed a commit that referenced this pull request Mar 8, 2017
…rd and project column (#544)

* #514: added inviter to Invitation for orgs and teams invite

* #514: go fmt

* #514: removed prints

* #514: changed time data types and included them in test

* #514 Added Org List Pending Invitations

* #514 removed typos in error handling

* #514: Modified to pointer time.Time in Invitation

* #514: Set up Naming conventions for multi words, pointer to bool

* #514: multi word variable format

* #514 : used User instead of new struct

* #514 : removed Inviter struct

* #514: Added quotes to list values in comment

* implementation of webhooks for project, projectcard and project column

* Added Changes in event types

* Added Comment for variables populated by webhook

* #530: Made all Changes struct inline

* #530: Modified Changes Structs to include the full names of Changes

* #530 : Review comments from gmlewis fixed

* #530 : Fixed Consistency issues with GitHub docs #562

* #530:generator code
bubg-dev pushed a commit to bubg-dev/go-github that referenced this pull request Jun 16, 2017
…#562)

There were various slight variations in the phrasing of comments with
the form:

	// GitHub API docs: https://developer.github.com/v3/...

This change makes them all consistent.

Also fix "GitHub" misspelled as "Github" in a few places.

Followup to google#551.
bubg-dev pushed a commit to bubg-dev/go-github that referenced this pull request Jun 16, 2017
…rd and project column (google#544)

* google#514: added inviter to Invitation for orgs and teams invite

* google#514: go fmt

* google#514: removed prints

* google#514: changed time data types and included them in test

* google#514 Added Org List Pending Invitations

* google#514 removed typos in error handling

* google#514: Modified to pointer time.Time in Invitation

* google#514: Set up Naming conventions for multi words, pointer to bool

* google#514: multi word variable format

* google#514 : used User instead of new struct

* google#514 : removed Inviter struct

* google#514: Added quotes to list values in comment

* implementation of webhooks for project, projectcard and project column

* Added Changes in event types

* Added Comment for variables populated by webhook

* google#530: Made all Changes struct inline

* google#530: Modified Changes Structs to include the full names of Changes

* google#530 : Review comments from gmlewis fixed

* google#530 : Fixed Consistency issues with GitHub docs google#562

* google#530:generator code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants