You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pr@flagon.apache.org by "brucearctor (via GitHub)" <gi...@apache.org> on 2026/10/12 01:54:36 UTC
[PR] chore: migrate Distill from Poetry to uv and tighten UserALE CI tooling (flagon)
brucearctor opened a new pull request, #195:
URL: https://github.com/apache/flagon/pull/195
## Summary
### 1. Distill: Migrate from Poetry to `uv` (PEP 621)
- **[products/distill/pyproject.toml](products/distill/pyproject.toml)**: Migrated from `[tool.poetry]` to standard PEP 621 `[project]` metadata (`requires-python = ">=3.10,<4.0"`), `[dependency-groups]` (`test`, `lint`, `docs`, `dev`), and `hatchling` build backend. Removed accidental PyPI `datetime` (Zope `DateTime`) dependency.
- **Lockfile**: Replaced `poetry.lock` with `uv.lock`.
- **[.github/workflows/distill_ci.yml](.github/workflows/distill_ci.yml)**: Consolidated the split Poetry (3.8–3.11) + `uv` (3.12–3.13) jobs into a single unified `astral-sh/setup-uv@v4` matrix across Python `3.10`, `3.11`, `3.12`, and `3.13` using `uv sync --locked --group test` and `uv run --no-sync pytest`.
- **[products/distill/Makefile](products/distill/Makefile) & [products/distill/README.md](products/distill/README.md)**: Updated targets (`sync`, `build`, `publish_pytest`, `test`, `lint`, `format`) and contributor instructions to use `uv`.
- **[lefthook.yml](lefthook.yml)**: Added `distill-lint` (`ruff check`) to `pre-commit` and `distill-test` (`pytest`) to `pre-push`.
### 2. UserALE: Node/JS Tooling & CI Improvements
- **[products/userale/package.json](products/userale/package.json) & [.github/workflows/userale_ci.yml](.github/workflows/userale_ci.yml)**: Added non-mutating `format:check` (`prettier --check`) and `lint:check` (`eslint`) scripts and wired them into `userale_ci.yml` (`pre-build`) so CI fails on unformatted/unlinted code instead of silently rewriting files on the runner. Removed dead `go env GOPATH` step and non-existent Node `26.x` matrix entry.
- **Formatted files caught by `pnpm format:check`**: `packages/flagon-userale-ext/src/background/index.ts` and `packages/flagon-userale/src/gen/apache/flagon/userale/v1/log_pb.ts`.
- **[products/userale/packages/flagon-userale/package.json](products/userale/packages/flagon-userale/package.json)**: Removed unused `cypress` devDependency (UserALE uses Playwright), eliminating 117 transitive dependencies.
- **[products/userale/eslint.config.js](products/userale/eslint.config.js) & [products/userale/packages/flagon-userale/src/main.ts](products/userale/packages/flagon-userale/src/main.ts)**: Disabled base ESLint `no-unused-vars` in favor of `@typescript-eslint/no-unused-vars` (fixing false warnings on TypeScript function type signatures) and removed duplicate unused imports in `main.ts`.
--
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: pr-unsubscribe@flagon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@flagon.apache.org
For additional commands, e-mail: pr-help@flagon.apache.org
Re: [PR] chore: migrate Distill from Poetry to uv and tighten UserALE CI tooling (flagon)
Posted by "brucearctor (via GitHub)" <gi...@apache.org>.
brucearctor merged PR #195:
URL: https://github.com/apache/flagon/pull/195
--
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: pr-unsubscribe@flagon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@flagon.apache.org
For additional commands, e-mail: pr-help@flagon.apache.org