Repository navigation
ci: use airbytehq forks of aaronsteers actions - #1302
Pedro S. Lopez (pedroslopez) wants to merge 1 commit into
Conversation
Co-Authored-By: Pedro Lopez <[email protected]>
|
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 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/1790879106-use-airbytehq-action-forks' pyairbyte --help
# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/airbytehq/PyAirbyte.git@devin/1790879106-use-airbytehq-action-forks'PR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful ResourcesCommunity SupportQuestions? Join the #pyairbyte channel in our Slack workspace. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| - name: Run Poe Slash Command Processor | ||
| uses: aaronsteers/[email protected] | ||
| uses: airbytehq/[email protected] |
There was a problem hiding this comment.
This ref isn't new. The same tag was used under aaronsteers/ before. This PR only changes the owner to the airbytehq fork, so the alert fires because the line counts as changed. Switching to SHA pins is out of scope here; I've raised it with the requester as a possible follow-up.
|
|
||
| - name: Create draft release (not ready) | ||
| uses: aaronsteers/[email protected] | ||
| uses: airbytehq/[email protected] |
There was a problem hiding this comment.
This ref isn't new. The same tag was used under aaronsteers/ before. This PR only changes the owner to the airbytehq fork, so the alert fires because the line counts as changed. Switching to SHA pins is out of scope here; I've raised it with the requester as a possible follow-up.
|
|
||
| - name: Attach assets and finalize draft | ||
| uses: aaronsteers/[email protected] | ||
| uses: airbytehq/[email protected] |
There was a problem hiding this comment.
This ref isn't new. The same tag was used under aaronsteers/ before. This PR only changes the owner to the airbytehq fork, so the alert fires because the line counts as changed. Switching to SHA pins is out of scope here; I've raised it with the requester as a possible follow-up.
Code Coverage OverviewLanguages: Python Python / code-coverage/pytest-no-credsThe overall line coverage in commit 074ffdc in the Show a line coverage summary of the most impacted files.
Python / code-coverage/pytestThe overall line coverage in commit 074ffdc in the Show a line coverage summary of the most impacted files.
Python / code-coverage/pytest-unitThe overall line coverage in commit 074ffdc in the Show a line coverage summary of the most covered files.
Updated |
This branch has not been deployed
2 participants
Summary
Points every usage of the
aaronsteers/{semantic-pr-release-drafter,tk-todo-check-action,poe-command-processor}actions at the new airbytehq forks (semantic-pr-release-drafter, tk-todo-check-action, poe-command-processor), so our CI no longer depends on a personal account.Only the owner changes,
aaronsteers/becomesairbytehq/. Every ref (tag, SHA pin,@main) stays exactly as it was. The forks carry the same tags, and the pinned SHAs resolve to the same commits on the forks. Any CONTRIBUTING/docs links to the drafter repo now point at the fork too.Requested by Pedro S. Lopez (@pedroslopez).
Link to Devin session: https://app.devin.ai/sessions/d51661a7e4c0485caae6e5b26d81c6a0
Open in Devin Desktop: https://app.devin.ai/desktop/session/d51661a7e4c0485caae6e5b26d81c6a0?variant=devin