-
Notifications
You must be signed in to change notification settings - Fork 67
feat: Add PR welcome message system with uvx installation examples #721
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
Conversation
- Add GitHub workflow to automatically comment on new PRs - Include community and internal welcome message templates - Provide uvx syntax for testing PyAirbyte from PR branches - Include links to #pyairbyte Slack channel and documentation - Add slash command documentation for PR automation Requested by @aaronsteers Co-Authored-By: AJ Steers <[email protected]>
Original prompt from AJ Steers:
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. Testing This PyAirbyte VersionYou can test this version of PyAirbyte using the following: # Run PyAirbyte CLI from this branch:
uvx 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1752801376-add-pr-welcome-message' pyab --help
# Or use the full command name:
uvx 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1752801376-add-pr-welcome-message' pyairbyte --help
# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1752801376-add-pr-welcome-message' Helpful ResourcesPR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
Community SupportQuestions? Join the #pyairbyte channel in our Slack workspace. |
- Replace hardcoded airbytehq/PyAirbyte with {{ .repo_name }} template variable - Ensures uvx commands work correctly for community forks - Addresses feedback from @aaronsteers on PR #721 Co-Authored-By: AJ Steers <[email protected]>
- Add --from flag to uvx commands as required when package name differs from CLI name - Addresses @aaronsteers feedback on PR #721 about incorrect uvx syntax - Follows PyAirbyte CLI documentation pattern: uvx --from=airbyte pyab --help Co-Authored-By: AJ Steers <[email protected]>
- Remove shorter 'pyab' syntax to save space in welcome message - Keep only the full 'pyairbyte' command as suggested by @aaronsteers - Addresses feedback on PR #721 about reducing template verbosity Co-Authored-By: AJ Steers <[email protected]>
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. Testing This PyAirbyte VersionYou can test this version of PyAirbyte using the following: # Run PyAirbyte CLI from this branch:
uvx --from 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1752801376-add-pr-welcome-message' pyairbyte --help
# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1752801376-add-pr-welcome-message' Helpful ResourcesPR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
Community SupportQuestions? Join the #pyairbyte channel in our Slack workspace. |
Add PR welcome message system with uvx installation examples
Summary
This PR implements an automated PR welcome message system for PyAirbyte, similar to the one used in the Python CDK. When new PRs are opened (especially from community forks), a GitHub workflow automatically posts a welcome comment with:
The implementation adds 3 new files:
Review & Testing Checklist for Human
uvx 'git+https://github.com/airbytehq/PyAirbyte.git@branch-name' pyab --help
/fix-pr
,/test-pr
, and/poetry-lock
commands actually exist and work as described{{ .repo_name }}
and{{ .branch_name }}
variables populate correctly in the rendered messagesDiagram
Notes
pyab
andpyairbyte
aliases)Link to Devin session: https://app.devin.ai/sessions/8a2585a98e604164a973493586dbe063
Requested by: @aaronsteers
Important
Auto-merge enabled.
This PR is set to merge automatically when all requirements are met.
Note
Auto-merge may have been disabled. Please check the PR status to confirm.