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

Add a "Reviewed by Hound" badge #7823

Merged
merged 1 commit into from
Feb 20, 2020
Merged

Conversation

salbertson
Copy link
Contributor

screenshot 2018-07-31 13 10 56

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@andrewvc
Copy link
Contributor

This is nice to add, but I suppose this doesn't link anywhere specific to the beats project? Is there an overview page for violations across the project?

@salbertson
Copy link
Contributor Author

@andrewvc you're right, it links to Hound's landing page. All of Hound's interaction and UI is directly in GitHub so there isn't anything beats specific to link to.

@salbertson
Copy link
Contributor Author

@andrewvc it's simply sharing that the project uses Hound, and a bit of marketing for a free-for-open-source service. What do you think?

@andresrc andresrc merged commit 9eeaf9a into elastic:master Feb 20, 2020
@salbertson
Copy link
Contributor Author

👏

@andresrc andresrc added the Team:Integrations Label for the Integrations team label Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants