Skip to content

ci: fix multipart regression lint failures - #379

Merged
etr merged 1 commit into
masterfrom
fix/ci-multipart-lint
Oct 8, 2026
Merged

etr merged 1 commit into
masterfrom
fix/ci-multipart-lint

Conversation

@etr

@etr etr commented Oct 7, 2026

Copy link
Copy Markdown
Owner

The raw multipart upload regression helper introduced three cpplint runtime/int failures, leaving Verify Build red. Add scoped annotations for the long types required by libcurl's POSTFIELDSIZE and response-code APIs, with a rationale matching the existing upload helpers.

Validation: reproduced the three CI failures twice before the change; the full recursive cpplint command now passes. Warning-suppression and file-size gates pass, and git diff --check is clean. This changes only comments and ChangeLog; executable test behavior is unchanged.

@codecov

codecov Bot commented Oct 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.51%. Comparing base (ae84459) to head (4a12d5d).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage   69.85%   69.51%   -0.34%     
==========================================
  Files          80       80              
  Lines        4229     4229              
  Branches     1521     1521              
==========================================
- Hits         2954     2940      -14     
- Misses        366      372       +6     
- Partials      909      917       +8     

see 13 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae84459...4a12d5d. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@etr
etr merged commit ff85cba into master Oct 8, 2026
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant