Skip to content

chore(qsl): phase2 compat 2026.07.1 + legacy req refs#114

Merged
Pigbibi merged 2 commits into
mainfrom
codex/qsl-compat-phase2-pipeline
Jul 3, 2026
Merged

chore(qsl): phase2 compat 2026.07.1 + legacy req refs#114
Pigbibi merged 2 commits into
mainfrom
codex/qsl-compat-phase2-pipeline

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Update qsl.compat.bundle to 2026.07.1
  • Keep legacy req/pyproject dependency refs aligned to current live commit set
  • keep allow_legacy=true and enforce_bundle=false
  • Sync requirements.txt, requirements-lock.txt and uv.lock
  • Update qsl metadata smoke test expectation

Validation

  • uv lock --check
  • uv run pytest tests/test_qsl_metadata.py -q
  • git diff --check

Co-Authored-By: Codex <noreply@openai.com>

@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: 35f22fdc57

ℹ️ 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 requirements.txt
@@ -1,4 +1,4 @@
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@0063af3b4a974650ea58a7d3f26dd1b94f65d3e8
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@7032cde4547e7ec59af15df8935d142461a77051

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Update the lock-file test for the new QPK revision

With this new revision, tests/test_monthly_publish_workflow_config.py still defines QPK_DEPENDENCY with the old 0063af... hash and test_real_publish_dependency_is_locked asserts that value is present in both requirements files; CI runs python -m unittest discover -s tests -v in .github/workflows/ci.yml, so this commit makes the unit test fail (confirmed with python -m unittest tests.test_monthly_publish_workflow_config -v). Please update the test fixture alongside this dependency bump.

Useful? React with 👍 / 👎.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi Pigbibi merged commit 9ea4eb3 into main Jul 3, 2026
2 checks passed
@Pigbibi Pigbibi deleted the codex/qsl-compat-phase2-pipeline branch July 3, 2026 10:20
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