Skip to content

Commit c682451

Browse files
Fix editorconfig errors: remove trailing whitespace
Remove trailing whitespace from: - .github/workflows/performance-regression.yml (15 lines) - docs/performance-regression-testing.md (1 line) - scripts/db-schema-migration.sql (4 lines) Addresses editorconfig check failures. Co-Authored-By: Alex Peng <[email protected]>
1 parent bbb7b3f commit c682451

File tree

3 files changed

+20
-20
lines changed

3 files changed

+20
-20
lines changed

.github/workflows/performance-regression.yml

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ env:
4848
jobs:
4949
performance-cpu:
5050
runs-on: ubuntu-latest
51-
51+
5252
steps:
5353
- name: Clone
5454
uses: actions/checkout@v4
@@ -59,7 +59,7 @@ jobs:
5959
uses: actions/setup-python@v5
6060
with:
6161
python-version: '3.x'
62-
62+
6363
- name: Install Python dependencies
6464
run: |
6565
pip install GitPython tabulate matplotlib
@@ -123,7 +123,7 @@ jobs:
123123
--current ${{ env.RESULTS_DB }} \
124124
--threshold ${{ env.REGRESSION_THRESHOLD }} \
125125
--output regression-report.md
126-
126+
127127
# Set output for subsequent steps
128128
if [ -f regression-detected.flag ]; then
129129
echo "regression=true" >> $GITHUB_OUTPUT
@@ -148,13 +148,13 @@ jobs:
148148
script: |
149149
const fs = require('fs');
150150
let report = 'Performance Regression Test Results (CPU)\n\n';
151-
151+
152152
if (fs.existsSync('regression-report.md')) {
153153
report += fs.readFileSync('regression-report.md', 'utf8');
154154
} else {
155155
report += 'No regression report generated.';
156156
}
157-
157+
158158
github.rest.issues.createComment({
159159
issue_number: context.issue.number,
160160
owner: context.repo.owner,
@@ -178,7 +178,7 @@ jobs:
178178
performance-cuda:
179179
runs-on: gpu-runner
180180
if: false # Disabled by default - enable when GPU runners are available
181-
181+
182182
steps:
183183
- name: Clone
184184
uses: actions/checkout@v4
@@ -189,7 +189,7 @@ jobs:
189189
uses: actions/setup-python@v5
190190
with:
191191
python-version: '3.x'
192-
192+
193193
- name: Install Python dependencies
194194
run: |
195195
pip install GitPython tabulate matplotlib
@@ -253,7 +253,7 @@ jobs:
253253
--current ${{ env.RESULTS_DB }} \
254254
--threshold ${{ env.REGRESSION_THRESHOLD }} \
255255
--output regression-report-cuda.md
256-
256+
257257
if [ -f regression-detected.flag ]; then
258258
echo "regression=true" >> $GITHUB_OUTPUT
259259
else
@@ -277,13 +277,13 @@ jobs:
277277
script: |
278278
const fs = require('fs');
279279
let report = 'Performance Regression Test Results (CUDA)\n\n';
280-
280+
281281
if (fs.existsSync('regression-report-cuda.md')) {
282282
report += fs.readFileSync('regression-report-cuda.md', 'utf8');
283283
} else {
284284
report += 'No regression report generated.';
285285
}
286-
286+
287287
github.rest.issues.createComment({
288288
issue_number: context.issue.number,
289289
owner: context.repo.owner,
@@ -306,7 +306,7 @@ jobs:
306306

307307
performance-metal:
308308
runs-on: macos-14 # macOS with Apple Silicon for Metal testing
309-
309+
310310
steps:
311311
- name: Clone
312312
uses: actions/checkout@v4
@@ -317,7 +317,7 @@ jobs:
317317
uses: actions/setup-python@v5
318318
with:
319319
python-version: '3.x'
320-
320+
321321
- name: Install Python dependencies
322322
run: |
323323
pip install GitPython tabulate matplotlib
@@ -382,7 +382,7 @@ jobs:
382382
--current ${{ env.RESULTS_DB }} \
383383
--threshold ${{ env.REGRESSION_THRESHOLD }} \
384384
--output regression-report-metal.md
385-
385+
386386
if [ -f regression-detected.flag ]; then
387387
echo "regression=true" >> $GITHUB_OUTPUT
388388
else
@@ -406,13 +406,13 @@ jobs:
406406
script: |
407407
const fs = require('fs');
408408
let report = 'Performance Regression Test Results (Metal)\n\n';
409-
409+
410410
if (fs.existsSync('regression-report-metal.md')) {
411411
report += fs.readFileSync('regression-report-metal.md', 'utf8');
412412
} else {
413413
report += 'No regression report generated.';
414414
}
415-
415+
416416
github.rest.issues.createComment({
417417
issue_number: context.issue.number,
418418
owner: context.repo.owner,

docs/performance-regression-testing.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ If the baseline cache is empty or expired:
289289
Regressions can be marked as false positives in the database:
290290

291291
```sql
292-
UPDATE regression_alerts
292+
UPDATE regression_alerts
293293
SET status = 'false_positive', notes = 'Expected due to architectural change'
294294
WHERE id = <alert_id>;
295295
```

scripts/db-schema-migration.sql

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ CREATE INDEX IF NOT EXISTS idx_memory_leak_logs_commit ON memory_leak_logs(build
7070
CREATE INDEX IF NOT EXISTS idx_memory_leak_logs_time ON memory_leak_logs(test_time);
7171

7272
CREATE VIEW IF NOT EXISTS latest_baselines AS
73-
SELECT
73+
SELECT
7474
b.baseline_name,
7575
b.commit_sha,
7676
b.branch_name,
@@ -83,7 +83,7 @@ GROUP BY b.id
8383
ORDER BY b.created_at DESC;
8484

8585
CREATE VIEW IF NOT EXISTS regression_summary AS
86-
SELECT
86+
SELECT
8787
current_commit,
8888
COUNT(*) as total_regressions,
8989
SUM(CASE WHEN severity = 'critical' THEN 1 ELSE 0 END) as critical_count,
@@ -95,7 +95,7 @@ GROUP BY current_commit
9595
ORDER BY total_regressions DESC;
9696

9797
CREATE VIEW IF NOT EXISTS memory_leak_summary AS
98-
SELECT
98+
SELECT
9999
build_commit,
100100
COUNT(*) as total_tests,
101101
SUM(CASE WHEN memory_status = 'LLAMA_MEMORY_STATUS_SUCCESS' THEN 1 ELSE 0 END) as passed_tests,
@@ -110,7 +110,7 @@ AFTER UPDATE ON regression_alerts
110110
FOR EACH ROW
111111
WHEN OLD.status != NEW.status
112112
BEGIN
113-
UPDATE regression_alerts
113+
UPDATE regression_alerts
114114
SET alert_time = datetime('now')
115115
WHERE id = NEW.id;
116116
END;

0 commit comments

Comments
 (0)