Skip to content

Commit fb0b04a

Browse files
authored
ci: fix qpk pin downstream ref check
Squash merge follow-up fix for update-qpk-pin downstream ref validation.
1 parent 68ee5ee commit fb0b04a

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

.github/workflows/update-qpk-pin.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,16 +65,24 @@ jobs:
6565
python -c "from quant_platform_kit.notifications.telegram import send_telegram_message; print('telegram OK')"
6666
python -c "from quant_platform_kit.common.contracts import SnapshotProfileContract; print('contracts OK')"
6767
68-
# Verify strategy repos installable with NEW constraints
68+
# Verify strategy repo refs are fetchable and package metadata is buildable.
69+
#
70+
# Strategy packages currently carry their own direct QPK pin. A full
71+
# dependency solve with the newly generated top-level QPK constraint
72+
# would conflict until downstream repos update those pins, so keep this
73+
# check focused on the package refs generated above.
6974
failed=0
7075
for dep in us-equity-strategies hk-equity-strategies cn-equity-strategies crypto-strategies; do
7176
echo "Checking $dep..."
72-
if python -m pip install --dry-run -c constraints.txt "$dep" >/dev/null 2>&1; then
77+
log_file="$(mktemp)"
78+
if python -m pip install --dry-run --no-deps -c constraints.txt "$dep" >"${log_file}" 2>&1; then
7379
echo " $dep OK"
7480
else
7581
echo " $dep FAILED"
82+
sed -n '1,160p' "${log_file}"
7683
failed=1
7784
fi
85+
rm -f "${log_file}"
7886
done
7987
if [ "${failed}" -ne 0 ]; then
8088
echo "One or more downstream dependency checks failed." >&2

0 commit comments

Comments
 (0)