Skip to content

chore: auto-update QPK_PIN and pin manifests - #291

Closed
Pigbibi wants to merge 1 commit into
mainfrom
auto/qpk-pin-update
Closed

chore: auto-update QPK_PIN and pin manifests#291
Pigbibi wants to merge 1 commit into
mainfrom
auto/qpk-pin-update

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Automated update of QPK_PIN and QSL Git SHA pin manifests.

Compatibility checks passed ✅ — all downstream-critical QPK modules verified importable.

Updated SHAs:

  • QPK: 5d4bbd0e7ef9a1434010e8b6a69905d39ee55f1b
  • Strategy repos: latest HEAD from each

🤖 Generated with Claude Code

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9aa879dfb1

ℹ️ 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".

Comment thread constraints.txt
Comment on lines +4 to +7
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@5d4bbd0e7ef9a1434010e8b6a69905d39ee55f1b
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@3101a4a04b954907e6d9fed21de935d0eab033f5
hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@d5875104381974a697b6a638b463f9cc083c4279
cn-equity-strategies @ git+https://github.com/QuantStrategyLab/CnEquityStrategies.git@fa5ac4b3f0fe4cad3a6c193cfbfe6380a6abd54d

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Align strategy pins before publishing the new QPK constraint

These strategy refs are not installable with the new QPK constraint: the added Us/Hk/Cn commits still declare quant-platform-kit @ ...@230854059c11a3711fed4bd997e3f8a7f37cba32 in their package metadata (US, HK, CN), while line 4 constrains QPK to 5d4bbd0e7ef9a1434010e8b6a69905d39ee55f1b. Since qsl.toml:8 marks constraints.txt as the live constraint file, any downstream install such as pip install -c constraints.txt us-equity-strategies has to satisfy two different direct QPK URLs and will fail resolution until these pins point at downstream commits updated to 5d4bbd0e7ef9a1434010e8b6a69905d39ee55f1b.

Useful? React with 👍 / 👎.

@Pigbibi

Pigbibi commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Closing unmerged: blocked unsatisfiable mixed pin graph; superseded by strategy-first QPK-5d4 bootstrap.

@Pigbibi Pigbibi closed this Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant