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

Doc: add an example for GitHub Actions #55

Merged
merged 1 commit into from
May 11, 2020

Conversation

rachelcarmena
Copy link
Contributor

Some missing documentation after integrating Hood into Arrow Fx library.

@rachelcarmena
Copy link
Contributor Author

Thanks @calvellido for the review 🙌

@rachelcarmena rachelcarmena merged commit 8173772 into master May 11, 2020
@rachelcarmena rachelcarmena deleted the add-github-actions-doc branch May 11, 2020 08:31
@@ -10,6 +10,8 @@ permalink: gradle/ci/
**Hood** gives you the ability to integrate the benchmark comparison into your `CI`
and integrate the result into a `Github` pull request.

It provides `compareBenchmarksCI` Gradle task which compares 2 reports from a benchmark and then it creates a new comment with the comparison result in the pull request. For instance, [look at this comment](https://github.com/47degrees/helios/pull/137#issuecomment-597753181). If new commits are added to the same pull request, the comment will be updated with the new comparison result.
Copy link
Contributor

Choose a reason for hiding this comment

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

actually compare a benchmarks with a series of benchmarks no just 2

@@ -10,6 +10,8 @@ permalink: gradle/ci/
**Hood** gives you the ability to integrate the benchmark comparison into your `CI`
and integrate the result into a `Github` pull request.

It provides `compareBenchmarksCI` Gradle task which compares 2 reports from a benchmark and then it creates a new comment with the comparison result in the pull request. For instance, [look at this comment](https://github.com/47degrees/helios/pull/137#issuecomment-597753181). If new commits are added to the same pull request, the comment will be updated with the new comparison result.
Copy link
Contributor

Choose a reason for hiding this comment

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

it also adds an status to the pull request

@AdrianRaFo
Copy link
Contributor

Apparently my review comments weren't posted

@rachelcarmena
Copy link
Contributor Author

Apparently my review comments weren't posted

Don't worry, I'll create another PR to fix it 🙌 Thanks for the review @AdrianRaFo !!

rachelcarmena added a commit that referenced this pull request May 11, 2020
rachelcarmena added a commit that referenced this pull request May 11, 2020
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.

3 participants