Skip to content

ci: replace per-PR #team-apis Slack post with a weekday ready-for-review digest - #1262

Merged
Patrick Nilan (pnilan) merged 1 commit into
mainfrom
devin/1793742847-slack-pr-daily-digest
Nov 3, 2026
Merged

Patrick Nilan (pnilan) merged 1 commit into
mainfrom
devin/1793742847-slack-pr-daily-digest

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Summary

#team-apis now gets one summary each weekday morning of python-cdk PRs that became ready for review, instead of one Slack message per PR. Same approach as airbytehq/airbyte#90873.

  • Deletes slack-pr-notification.yml (the pull_request_target post on opened / ready_for_review).
  • Adds slack-pr-daily-digest.yml:
    • Schedule: cron: "0 8 * * 1-5", timezone: America/Los_Angeles.
    • Window: previous day 08:00 PT until the run. On Mondays it starts Friday 08:00 PT (72h), so weekend PRs are included.
    • Query: GraphQL search repo:… is:pr is:open draft:false updated:>=<start>, then keeps PRs whose ready time is in the window:
      readyAt = last READY_FOR_REVIEW_EVENT.createdAt ?? createdAt
      
      So it covers both cases the old workflow posted for: opened as non-draft, and draft marked ready.
    • Only PRs still open and non-draft at digest time are listed. PRs merged or closed inside the window are dropped, because nobody needs to review them.
    • Output: Slack mrkdwn text, not blocks, so a long Monday list can't hit the 3,000-character block limit. Titles and author logins are escaped. <!subteam^S077U1TH43D> is mentioned only when the list isn't empty.
    • The workflow_dispatch dry-run input writes the digest to the job summary without posting it.
    • Moving off pull_request_target means no workflow runs with the Slack token in a PR context anymore.

Example (rendered locally against live data with the window widened to Oct 1):

:mag: *PRs Ready for Review: Daily Summary* in `airbytehq/airbyte-python-cdk`
Tuesday, November 3, 2026 at 8:00 AM PT
<!subteam^S077U1TH43D> *2 PRs* became ready for review in the last 24 hours:
• <https://github.com/airbytehq/airbyte-python-cdk/pull/1172|#1172> feat(file-based): declare Unstructured Document Parser `content` as semantic-search indexable — by <https://github.com/aaronsteers|aaronsteers>  ·  `+63` `-5` across 5 file(s)
• <https://github.com/airbytehq/airbyte-python-cdk/pull/1246|#1246> fix(declarative): copy the manifest once when propagating types and parameters — by <https://github.com/apps/devin-ai-integration|devin-ai-integration>  ·  `+373` `-17` across 6 file(s)

(#1172 was created on Sep 23 and marked ready on Oct 15. It sorts by its ready time, which shows the ready-for-review event is used.)

Testing

  • actionlint and prettier --check pass.
  • Ran each step's script locally against live GitHub data without posting. Today's 24h window gives the "No PRs" message with no mention. The widened window gives the output above.
  • Monday math: date -d "2026-11-09 08:00 3 days ago" gives Friday 2026-11-06, start 2026-11-06T16:00:00Z.
  • Not run in Actions yet. workflow_dispatch only works once the file is on main. After merge, a dry-run=true dispatch is the first end-to-end check.

Link to Devin session: https://app.devin.ai/sessions/e378de9fd9d949439935a901d2158595
Open in Devin Desktop: https://app.devin.ai/desktop/session/e378de9fd9d949439935a901d2158595?variant=devin

Important

Auto-merge enabled.

This PR is set to merge automatically when all requirements are met.

… digest

Co-Authored-By: patrick.nilan@airbyte.io <patrick.nilan@airbyte.io>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

Testing This CDK Version

You can test this version of the CDK using the following:

# Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/airbyte-python-cdk.git@devin/1793742847-slack-pr-daily-digest#egg=airbyte-python-cdk[dev]' --help

# Update a connector to use the CDK from this branch ref:
cd airbyte-integrations/connectors/source-example
poe use-cdk-branch devin/1793742847-slack-pr-daily-digest

PR Slash Commands

Airbyte Maintainers can execute the following slash commands on your PR:

  • /autofix - Fixes most formatting and linting issues
  • /poetry-lock - Updates poetry.lock file
  • /test - Runs connector tests with the updated CDK
  • /prerelease - Triggers a prerelease publish with default arguments
  • /poe build - Regenerate git-committed build artifacts, such as the pydantic models which are generated from the manifest JSON schema in YAML.
  • /poe <command> - Runs any poe command in the CDK environment
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

@pnilan
Patrick Nilan (pnilan) marked this pull request as ready for review November 3, 2026 21:56
@pnilan
Patrick Nilan (pnilan) requested a review from a team as a code owner November 3, 2026 21:56
@pnilan
Patrick Nilan (pnilan) enabled auto-merge (squash) November 3, 2026 21:56
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

PyTest Results (Fast)

6 306 tests  ±0   6 285 ✅ +1   6m 7s ⏱️ +37s
    1 suites ±0      21 💤  - 1 
    1 files   ±0       0 ❌ ±0 

Results for commit 03b036d. ± Comparison against base commit bbf1730.

@pnilan
Patrick Nilan (pnilan) merged commit aebf73c into main Nov 3, 2026
27 checks passed
@pnilan
Patrick Nilan (pnilan) deleted the devin/1793742847-slack-pr-daily-digest branch November 3, 2026 22:03
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