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 cmds #9

Merged
merged 20 commits into from
May 14, 2024
Merged

Add cmds #9

merged 20 commits into from
May 14, 2024

Conversation

pmaslana
Copy link
Contributor

@pmaslana pmaslana commented May 3, 2024

This is just an iterative step to create code needed for a keybase bot to send alerts for stale or pending PRs. Here are the conditions requested from Gene:

  1. If a PR has a new push from an outside contributor and no one has allowed the workflow run to start in say 2 hours - it would post an alert in a Keybase channel.

  2. If a community PR has no update from us in 7 days - also post an alert? Rules around this subject to some more thought but.

@pmaslana pmaslana marked this pull request as draft May 3, 2024 19:07
@pmaslana pmaslana marked this pull request as ready for review May 3, 2024 20:06
Copy link
Contributor

@Starttoaster Starttoaster left a comment

Choose a reason for hiding this comment

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

In addition to my other comments, maybe also do a go get -u ./... && go mod tidy, the dependency review job is upset https://github.com/Chia-Network/github-bot/actions/runs/8944275086/job/24570869218#step:3:7

k8s/notify_stale_pending_prs.yml.j2 Outdated Show resolved Hide resolved
internal/github/checkPendingCI.go Outdated Show resolved Hide resolved
internal/github/checkStalePRs.go Outdated Show resolved Hide resolved
internal/github/pullRequests.go Outdated Show resolved Hide resolved
cmd/notifyStale.go Outdated Show resolved Hide resolved
@pmaslana pmaslana dismissed cmmarslender’s stale review May 14, 2024 14:20

This was resolved

@pmaslana pmaslana merged commit c338f1e into main May 14, 2024
5 checks passed
@pmaslana pmaslana deleted the add-cmds branch May 14, 2024 17:27
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