[codex] Bump us equity strategies to 0.7.52#136
Conversation
![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: 9030123020
ℹ️ 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".
| gunicorn | ||
| quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@v0.7.36 | ||
| us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@v0.7.50 | ||
| us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@v0.7.52 |
There was a problem hiding this comment.
With this new pin, the Smoke import pinned shared packages step in .github/workflows/ci.yml lines 61-72 still runs resolve_canonical_profile("tech_communication_pullback_enhancement") against the package installed from requirements.txt before the editable checkout, but us-equity-strategies v0.7.52 removed that profile from its strategy catalog. PR/push CI will fail in that smoke step whenever it installs this requirements file; update the workflow assertion or keep the old pin until the smoke check is aligned.
Useful? React with 👍 / 👎.
Pigbibi
force-pushed
the
codex/bump-us-equity-strategies-0.7.52
branch
2 times, most recently
from
9e8433f to
b25288d
Compare
Pigbibi
force-pushed
the
codex/bump-us-equity-strategies-0.7.52
branch
from
b25288d to
bbf8e0d
Compare
1 participant
Summary
us-equity-strategiesdependency tov0.7.52Validation
git diff --check