Repository navigation
ci: replace per-PR #team-apis Slack post with a weekday ready-for-review digest - #1262
Merged
Patrick Nilan (pnilan) merged 1 commit intoNov 3, 2026
Merged
Conversation
… digest Co-Authored-By: patrick.nilan@airbyte.io <patrick.nilan@airbyte.io>
Contributor
Author
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksTesting This CDK VersionYou 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-digestPR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
|
Patrick Nilan (pnilan)
approved these changes
Nov 3, 2026
Patrick Nilan (pnilan)
marked this pull request as ready for review
November 3, 2026 21:56
Patrick Nilan (pnilan)
enabled auto-merge (squash)
November 3, 2026 21:56
Patrick Nilan (pnilan)
deleted the
devin/1793742847-slack-pr-daily-digest
branch
November 3, 2026 22:03
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.
slack-pr-notification.yml(thepull_request_targetpost onopened/ready_for_review).slack-pr-daily-digest.yml:cron: "0 8 * * 1-5",timezone: America/Los_Angeles.repo:… is:pr is:open draft:false updated:>=<start>, then keeps PRs whose ready time is in the window: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.workflow_dispatchdry-runinput writes the digest to the job summary without posting it.pull_request_targetmeans 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):
(#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
actionlintandprettier --checkpass.date -d "2026-11-09 08:00 3 days ago"gives Friday 2026-11-06, start2026-11-06T16:00:00Z.workflow_dispatchonly works once the file is onmain. After merge, adry-run=truedispatch 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.