Skip to content

perf(github): poke linked PRs only when it changes something - #152

Open
justanotherariel wants to merge 1 commit into
mainfrom
justanotherariel-smarter-pokes
Open

justanotherariel wants to merge 1 commit into
mainfrom
justanotherariel-smarter-pokes

Conversation

@justanotherariel

Copy link
Copy Markdown
Member

Proposed change

Pokes did work that nothing uses now that only docs and brands pair with code.

  • A review no longer reads or pokes linked PRs.
  • An edit that leaves the description alone pokes nothing.
  • A close or reopen only pokes the PRs that follow this one; the code side doesn't read its followers' state.
  • A poke is skipped while a run for the target is already waiting, and each PR takes at most 10 pokes per 30 minutes, so one PR listing many others can't drain the installation's rate limit. The limiter is shared with update's cap.

Type of change

  • Dependency upgrade
  • Bugfix
  • New rule or command
  • New repository manifest (enable the bot on another repo)
  • New feature (which adds functionality to an existing engine)
  • Code quality improvement or refactor (incl. tests)

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:

Checklist

  • I understand the code I am submitting and can explain how it works.
  • The code change is tested and works locally.
  • Local tests pass (mise run test). Your PR cannot be merged unless tests pass.
  • Lint and type checks pass (mise run check).
  • The code has been formatted (mise run format).
  • Tests have been added to verify that the new code works.
  • There is no commented out or debug code in this PR.
  • Any generated code has been carefully reviewed for correctness and compliance with project standards.

If behaviour, configuration, or environment variables changed:

  • README.md and .env.example have been updated accordingly.

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.

1 participant