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

ARROW-8064: [Dev] Implement Comment bot via Github actions #6571

Closed
wants to merge 33 commits into from

Conversation

kszucs
Copy link
Member

@kszucs kszucs commented Mar 10, 2020

See it in action: kszucs#16 (comment)

Main drawback that is is much slower than ursabot, but we can optimize it by:

  • porting crossbow to only depend on pygithub instead of libgit2 (will consume the rate limit, but should fit in)
  • use caching or docker

Theoretically CROSSBOW_GITHUB_TOKEN is set as a github actions secret, see https://issues.apache.org/jira/browse/INFRA-19954
We can trigger a build once this is merged into master.

kszucs added 27 commits March 5, 2020 12:19

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs
cat

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs
@github-actions
Copy link

kszucs added 2 commits March 11, 2020 17:48

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs
kszucs added 2 commits March 11, 2020 19:18

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs
RAT

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs
kszucs added 2 commits March 12, 2020 17:46

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs

Verified

This commit was signed with the committer’s verified signature. The key has expired.
kszucs Krisztián Szűcs
Copy link
Member Author

@kszucs kszucs left a comment

Choose a reason for hiding this comment

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

+1, the RTools build error is transient and unrelated.

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

2 participants