Skip to content

Add Python setup and aam-cli installation in CI workflow #2

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.

Sign up for GitHub

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

Merged
spazyCZ merged 1 commit into from
Feb 12, 2026
Merged

Add Python setup and aam-cli installation in CI workflow #2

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions .github/workflows/publish-pypi-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,17 @@ jobs:
- name: Install npm dependencies
run: npm ci

- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: "3.11"
cache: "pip"
cache-dependency-path: apps/aam-cli/pyproject.toml

- name: Install aam-cli with dev dependencies
run: pip install -e ".[dev]"
working-directory: apps/aam-cli
Comment on lines +57 to +58
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This workflow hard-codes the pip install -e ... command even though the repo already defines an aam-cli:install Nx target that does the same thing. To avoid drift if the install command changes (e.g., switching to uv/poetry or adjusting extras), consider invoking the Nx target here (or otherwise centralizing the install command).

Suggested change
run: pip install -e ".[dev]"
working-directory: apps/aam-cli
run: npx nx run aam-cli:install

Copilot uses AI. Check for mistakes.

- name: Lint and test aam-cli
run: npx nx run-many -t lint,test -p aam-cli

Expand Down
Loading