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.

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

Gitlab notifications #1141

Merged
merged 1 commit into from
Aug 21, 2015
Merged

Conversation

kpacha
Copy link

@kpacha kpacha commented Aug 16, 2015

send build status updates to gitlab as comments on the related commit

StatusPending = ":raised_hand:"
StatusSuccess = ":thumbsup:"
StatusFailure = ":thumbsdown:"
StatusError = ":exclamation:"

Choose a reason for hiding this comment

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

Add 'Started/Running' status?

@kpacha
Copy link
Author

kpacha commented Aug 17, 2015

@chengweiv5: the Started/Running status is not implemented because this plugin doesn't update the gitlab CI_Server status (https://gitlab.com/gitlab-org/gitlab-ce/blob/master/app/models/project_services/ci_service.rb#L44). It just sends a comment and I think it would be too verbose.

For an actual integration, I'd rather build a new gitlab CI_Service.

@bradrydzewski
Copy link

cool, merging this now, but just a heads up that 0.4-alpha.1 is going to release next week with a new plugin model and this will need to be manually ported over

bradrydzewski added a commit that referenced this pull request Aug 21, 2015
@bradrydzewski bradrydzewski merged commit 7347be7 into harness:master Aug 21, 2015
@kpacha kpacha deleted the gitlab_notifications branch August 21, 2015 07:21
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.

None yet

3 participants