Skip to content
This repository has been archived by the owner on Jun 8, 2019. It is now read-only.

Status-API #49

Merged
merged 3 commits into from
Mar 22, 2017
Merged

Status-API #49

merged 3 commits into from
Mar 22, 2017

Conversation

bkcsoft
Copy link
Member

@bkcsoft bkcsoft commented Mar 19, 2017

https://developer.github.com/v3/repos/statuses/

Fully decorated. Though I added status="warning" since I have a few ideas around that 😈

Implements API-side of go-gitea/gitea#357

@bkcsoft bkcsoft added this to the v1.2.0 milestone Mar 19, 2017
@bkcsoft bkcsoft mentioned this pull request Mar 19, 2017
12 tasks
@bkcsoft bkcsoft changed the title Initial support for Status-API Status-API Mar 19, 2017
@@ -0,0 +1,74 @@
package gitea
Copy link
Member

Choose a reason for hiding this comment

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

comment header

gitea/status.go Outdated
"time"
)

type StatusState string
Copy link
Member

Choose a reason for hiding this comment

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

lint

@bkcsoft
Copy link
Member Author

bkcsoft commented Mar 22, 2017

@lunny Done :)

Look maa'! No hands!
@lunny
Copy link
Member

lunny commented Mar 22, 2017

It's ready to review?

@bkcsoft
Copy link
Member Author

bkcsoft commented Mar 22, 2017

@lunny yes, should be. Removing WIP-flag now :)

@lunny
Copy link
Member

lunny commented Mar 22, 2017

LGTM

@appleboy
Copy link
Member

LGTM

@lunny lunny merged commit 1bec42c into master Mar 22, 2017
@tboerger tboerger deleted the bkcsoft/status-api branch March 28, 2017 14:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants