Skip to content

Commit c9d3167

Browse files
authored
Merge pull request #3 from manascb1344/refactor/modular-restructure
refactor: split monolithic app.py into runner/ package
2 parents c1de221 + ca5b351 commit c9d3167

21 files changed

Lines changed: 1147 additions & 985 deletions

.github/workflows/ci-cd.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,10 @@ on:
55
branches:
66
- main
77
paths:
8-
- 'src/**'
8+
- 'runner/**'
9+
- 'app.py'
910
- 'tests/**'
10-
- '*.py'
11-
- 'requirements.txt'
1211
- 'pyproject.toml'
13-
- 'Dockerfile'
1412
- '.github/workflows/ci-cd.yml'
1513
tags:
1614
- 'v*'
@@ -44,7 +42,7 @@ jobs:
4442
run: pytest tests/ -v
4543

4644
- name: Lint with ruff
47-
run: ruff check app.py
45+
run: ruff check runner/ app.py tests/
4846

4947
- name: Deploy job
5048
run: |

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ test:
1010
pytest tests/ -v
1111

1212
lint:
13-
ruff check app.py
13+
ruff check runner/ app.py tests/
1414

1515
logs:
1616
modal app logs modal-github-runner

0 commit comments

Comments
 (0)