Skip to content

Commit 4d5e9fe

Browse files
Add comments
1 parent 501f474 commit 4d5e9fe

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

.github/workflows/python-tests.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ jobs:
2121
steps:
2222
- uses: actions/checkout@v4
2323

24+
# Load dependencies from GitHub runner cache for quicker install
2425
- name: Set up Python ${{ matrix.python-version }}
2526
uses: actions/setup-python@v5
2627
with:
@@ -33,6 +34,7 @@ jobs:
3334
python -m pip install --upgrade pip
3435
pip install -r requirements.txt
3536
37+
# Lint the Python files & upload the result statistics
3638
- name: Run pylint analysis
3739
id: pylint
3840
continue-on-error: true
@@ -48,6 +50,7 @@ jobs:
4850
name: pylint-reports-${{ matrix.python-version }}
4951
path: tests/python/pylint/reports/
5052

53+
# Static code analysis through simple compilation to ensure code is syntactically sound
5154
- name: Verify bytecode compilation
5255
run: |
5356
python -m compileall infrastructure samples setup shared tests
@@ -75,6 +78,7 @@ jobs:
7578
name: junit-results-${{ matrix.python-version }}
7679
path: tests/python/junit-${{ matrix.python-version }}.xml
7780

81+
# Extract all linting and coverage results in preparation for publish
7882
- name: Extract and Summarize Metrics
7983
if: always()
8084
id: metrics
@@ -100,6 +104,7 @@ jobs:
100104
echo "coverage=N/A" >> "$GITHUB_OUTPUT"
101105
fi
102106
107+
# Publish general statistics for linting, test success, and code coverage as well as detailed tests results
103108
- name: Publish Consolidated Results to PR
104109
if: always() && github.event_name == 'pull_request'
105110
uses: marocchino/sticky-pull-request-comment@v2

0 commit comments

Comments
 (0)