Skip to content

Commit e1127f7

Browse files
committed
Merge requirements files
These were previously separate to enforce the idea that the scoring logic shouldn't depend on srcomp. However since we want to allow for the scorer UI to be defined in the compstate and to have tests, it's impractical to approach it this way. Merging these does allow some risk of an unexpected dependency, however it's probably a small enough risk not to be troublesome.
1 parent 4a1ca6e commit e1127f7

3 files changed

Lines changed: 3 additions & 7 deletions

File tree

action.yml

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,9 @@ runs:
2020
cache: pip
2121
cache-dependency-path: ${{ steps.copy-requirements.outputs.directory }}/**requirements.txt
2222

23-
- name: Install Test Dependencies
23+
- name: Install Dependencies
2424
shell: bash
25-
run: pip install -U -r ${{ steps.copy-requirements.outputs.directory }}/test-requirements.txt
25+
run: pip install -U -r ${{ steps.copy-requirements.outputs.directory }}/requirements.txt
2626

2727
- name: Run Scoring tests
2828
shell: bash
@@ -34,10 +34,6 @@ runs:
3434
python3 -m unittest discover --buffer scoring/tests/
3535
fi
3636
37-
- name: Install Dependencies
38-
shell: bash
39-
run: pip install -U -r ${{ steps.copy-requirements.outputs.directory }}/requirements.txt
40-
4137
- name: Run Compstate Validation
4238
shell: bash
4339
run: ${{ github.action_path }}/validate.py .

requirements.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1+
PyYAML>5,<7
12
sr.comp>=1.4,<2

test-requirements.txt

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)