Skip to content

Commit 351d193

Browse files
committed
ci: add ruff validation for longbridge platform
1 parent 573ee5d commit 351d193

4 files changed

Lines changed: 11 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,12 @@ jobs:
2222
set -euo pipefail
2323
python -m pip install --upgrade pip
2424
python -m pip install -r requirements.txt
25+
python -m pip install ruff
26+
27+
- name: Run ruff
28+
run: |
29+
set -euo pipefail
30+
ruff check .
2531
2632
- name: Run unit tests
2733
run: |

.ruff.toml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
target-version = "py311"
2+
3+
[lint.per-file-ignores]
4+
"tests/*.py" = ["E402"]

runtime_config_support.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,7 @@
55
from typing import Callable
66

77
from strategy_registry import (
8-
DEFAULT_STRATEGY_PROFILE,
98
LONGBRIDGE_PLATFORM,
10-
SUPPORTED_STRATEGY_PROFILES,
119
resolve_strategy_definition,
1210
)
1311

strategy_registry.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
from us_equity_strategies import get_strategy_definitions as get_us_equity_strategy_definitions
44

55
from quant_platform_kit.common.strategies import (
6+
StrategyDefinition,
67
US_EQUITY_DOMAIN,
78
get_supported_profiles_for_platform as qpk_get_supported_profiles_for_platform,
89
resolve_strategy_definition as qpk_resolve_strategy_definition,

0 commit comments

Comments
 (0)