Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: add support for github status checks #128

Merged
merged 1 commit into from
Jun 29, 2019

Conversation

andrewrynhard
Copy link
Member

Signed-off-by: Andrew Rynhard andrew@andrewrynhard.com

Signed-off-by: Andrew Rynhard <andrew@andrewrynhard.com>
// SetStatus sets the status of a GitHub check.
// Valid statuses are "error", "failure", "pending", "success"
func (c *Conform) SetStatus(state, policy, check, message string) {
if c.token == "" {
Copy link
Member Author

Choose a reason for hiding this comment

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

This is a cheesy way to make this optional for now. We have more immediate use cases and I can circle back and refactor. I'm thinking of a Summarizer interface that can be GitHub, PlainText,etc..

@andrewrynhard andrewrynhard merged commit 3be1319 into siderolabs:master Jun 29, 2019
@andrewrynhard andrewrynhard deleted the github-action branch August 7, 2019 20:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant