chore: auto-update QPK_PIN and pin manifests - #229
Conversation
🤖 Codex PR Review✅ Merge allowed: No issues found Review by Codex PR Review bot • PR |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d8bc2e707f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| cn-equity-strategies @ git+https://github.com/QuantStrategyLab/CnEquityStrategies.git@5b1a3da5e161d191e80193fa88a9905ec859c3f5 | ||
| crypto-strategies @ git+https://github.com/QuantStrategyLab/CryptoStrategies.git@8a9ec7f921763904b7d272f39627ac7bced4f1eb | ||
| quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@335c7a22bc3f570bd5705427ccc40172eda6b289 | ||
| us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@d545bced0dfe71406adc23c90dad7a2629cda14f |
There was a problem hiding this comment.
In environments that install constraints.txt as the live bundle (qsl.toml lists it under live_constraint_files), this new manifest is not resolvable: the pinned us-equity-strategies commit still declares a direct quant-platform-kit dependency at 31892425... in its pyproject, while line 4 pins QPK to 335c7a22.... The update workflow documents that a full dependency solve would conflict and only runs pip install --dry-run --no-deps (.github/workflows/update-qpk-pin.yml:75-84), so consumers installing this strategy with the generated constraints will hit a resolver conflict until the downstream strategy pin PRs are merged and this file is regenerated.
Useful? React with 👍 / 👎.
1 participant
Automated update of QPK_PIN and QSL Git SHA pin manifests.
Compatibility checks passed ✅ — all downstream-critical QPK modules verified importable.
Updated SHAs:
335c7a22bc3f570bd5705427ccc40172eda6b289🤖 Generated with Claude Code