Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Doc comments on Issue.Url and Issue.HtmlUrl are wrong/missing #875

Closed
Eilon opened this issue Aug 30, 2015 · 2 comments
Closed

Doc comments on Issue.Url and Issue.HtmlUrl are wrong/missing #875

Eilon opened this issue Aug 30, 2015 · 2 comments

Comments

@Eilon
Copy link
Contributor

Eilon commented Aug 30, 2015

This one is wrong (it says milestone instead of issue): https://github.com/octokit/octokit.net/blob/master/Octokit/Models/Response/Issue.cs#L34-L37

And this one is missing: https://github.com/octokit/octokit.net/blob/master/Octokit/Models/Response/Issue.cs#L39
@Eilon Eilon mentioned this issue Aug 30, 2015
Merged
haacked added a commit that referenced this issue Sep 1, 2015
@haacked
Merge pull request #876 from Eilon/patch-1
a90d3b8 
#875 fixing doc comments on Issue type
@mikeparker
Copy link

If the related PR is merged can this issue be closed? @Eilon

@shiftkey
Copy link
Member

shiftkey commented Sep 7, 2015

:shipit:

@shiftkey shiftkey closed this as completed Sep 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Eilon @shiftkey @mikeparker