ammachado commented on code in PR #25598:
URL: https://github.com/apache/camel/pull/25598#discussion_r4010811686
##########
.github/actions/incremental-build/action.yaml:
##########
@@ -41,19 +41,31 @@ inputs:
description: 'Extra Maven arguments to pass to the build (e.g.
-Denforcer.phase=none)'
required: false
default: ''
+ flake-label:
+ description: 'Names the matrix entry in the recovered-flake report (e.g.
"JDK 17"), so a reader can tell which entry a reported flake came from'
+ required: false
+ default: ''
runs:
using: "composite"
steps:
- id: install-mvnd
uses: apache/camel/.github/actions/install-mvnd@main
with:
dry-run: ${{ inputs.skip-mvnd-install }}
+ # collect-flakes.py declares its dependencies inline (PEP 723); uv is what
+ # honours that block. Installed here rather than in the caller so the
action
+ # stays self-contained for downstream repos that reuse it.
+ - id: install-uv
+ uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d #
v10.0.1
Review Comment:
@apupier, following up on this one: does the PEP 723 reasoning above settle
it for you, or would you rather use `actions/setup-python` plus an explicit
`pip install defusedxml` step? I'm happy to switch if you think the extra tool
isn't worth it. Both the uv and Python versions are now pinned, per your other
comment.
_Claude Code on behalf of Adriano Machado (@ammachado)_
_This was generated by an AI agent and may contain inaccuracies. Please
verify before relying on it._
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
