docs: split bilingual open-source readmes - #131
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 61c9743d1a
ℹ️ 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".
| python scripts/print_strategy_switch_env_plan.py --profile hk_listed_global_etf_rotation --account-region hk --dry-run-only --deployment-selector hk-verify --account-scope hk-verify --service-name longbridge-quant-hk-verify-service --json | ||
| gh workflow run sync-cloud-run-env.yml --repo QuantStrategyLab/LongBridgePlatform -f target=hk-verify -f cloud_run_region=<gcp-region> -f cloud_run_service=longbridge-quant-hk-verify-service -f longport_secret_name=longport_token_hk -f longport_app_key_secret_name=longport-app-key-hk -f longport_app_secret_secret_name=longport-app-secret-hk -f deploy_image=true -f sync_env=true | ||
| python -m pip install -r requirements.txt | ||
| python -m pytest -q |
There was a problem hiding this comment.
Add the test runner to the fresh-clone setup
In a fresh clone, the preceding command installs only requirements.txt, but that file does not include pytest while CI installs it separately before running the suite. As written, users in a clean Python environment will hit No module named pytest at this step instead of running tests; the mirrored Chinese quick start has the same issue. Please either install pytest here or use the repository's unittest command from CONTRIBUTING.md.
Useful? React with 👍 / 👎.
Summary
Validation