Skip to content

[yaml] - include yaml and tfrecord extras in py311-cloudcoverage - #40912

Draft
derrickaw wants to merge 1 commit into
apache:masterfrom
derrickaw:20261030_increaseYamlCodeCoverage
Draft

derrickaw wants to merge 1 commit into
apache:masterfrom
derrickaw:20261030_increaseYamlCodeCoverage

Conversation

@derrickaw

@derrickaw derrickaw commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

Helps with #39125

@github-actions github-actions Bot added python size/xs XS: <10 lines changed labels Oct 30, 2026
@codecov

codecov Bot commented Oct 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.12%. Comparing base (12bbb17) to head (da4865a).
⚠️ Report is 12 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #40912      +/-   ##
==========================================
+ Coverage   79.83%   80.12%   +0.29%     
==========================================
  Files         494      494              
  Lines       83575    83582       +7     
==========================================
+ Hits        66722    66971     +249     
+ Misses      16853    16611     -242     
Flag Coverage Δ
python 80.12% <ø> (+0.29%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@derrickaw

Copy link
Copy Markdown
Collaborator Author

/gemini review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Review Summary

This PR adds the tfrecord and yaml extras to the py311-cloudcoverage tox environment in sdks/python/tox.ini. This aligns py311-cloudcoverage with the default [testenv] extras configuration, ensuring that tests dependent on crcmod, quickjs-ng, and jsonschema are executed and measured during Python code coverage collection.

🔍 General Feedback

  • The change is clean, minimal, and free of dependency conflicts with existing gcp and cloudcoverage dependencies.
  • Codecov reports confirm the expected increase in Python test coverage (+249 hits).

💡 Tip: Comment /gemini review on this pull request to trigger a new Gemini code review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python size/xs XS: <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant