You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "jason810496 (via GitHub)" <gi...@apache.org> on 2026/07/23 06:40:28 UTC

[PR] Mount ts-sdk sources in Breeze container [airflow]

jason810496 opened a new pull request, #70278:
URL: https://github.com/apache/airflow/pull/70278

   - related: #69295
   
   ## Why
   
   `ts-sdk/` is the only language-SDK distribution missing from Breeze's `VOLUMES_FOR_SELECTED_MOUNTS` allowlist and the `.dockerignore` build-context allowlist, so unlike `task-sdk`, `go-sdk`, and `java-sdk` it never appears under `/opt/airflow` in Breeze shells (default `MOUNT_SELECTED` mode) or in the CI image.
   
   ## What
   
   - Add `ts-sdk` to `VOLUMES_FOR_SELECTED_MOUNTS` in `docker_command_utils.py` so it is bind-mounted at `/opt/airflow/ts-sdk`.
   - Add `!ts-sdk/` to the `.dockerignore` allowlist for parity with the other SDK distributions (`**/node_modules` stays excluded).
   - Regenerate `scripts/ci/docker-compose/local.yml` via `breeze setup synchronize-local-mounts`.
   
   ## Verification
   
   ```console
   $ breeze run ls /opt/airflow/ts-sdk
   LICENSE
   NOTICE
   README.md
   eslint.config.js
   example
   package.json
   pnpm-lock.yaml
   scripts
   src
   tests
   tsconfig.build.json
   tsconfig.json
   vitest.config.ts
   ```
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [x] Yes, with help of Claude Code Fable 5 following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Mount ts-sdk sources in Breeze container [airflow]

Posted by "jason810496 (via GitHub)" <gi...@apache.org>.
jason810496 merged PR #70278:
URL: https://github.com/apache/airflow/pull/70278


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org