Skip to content

ci: add automation folder to coverage reporting#3163

Merged
MarkusNeusinger merged 1 commit intomainfrom
ci/add-automation-coverage
Jan 2, 2026
Merged

ci: add automation folder to coverage reporting#3163
MarkusNeusinger merged 1 commit intomainfrom
ci/add-automation-coverage

Conversation

@MarkusNeusinger
Copy link
Copy Markdown
Owner

Summary

  • Add automation/ to pytest coverage measurement in CI pipeline
  • Previously only core/ and api/ were tracked

Change

- --cov=core --cov=api \
+ --cov=core --cov=api --cov=automation \

This enables tracking of test coverage for:

  • automation/generators/plot_generator.py
  • automation/scripts/sync_to_postgres.py
  • automation/scripts/workflow_utils.py
  • automation/scripts/label_manager.py

Test plan

  • CI workflow syntax is valid
  • Coverage report includes automation/ folder

🤖 Generated with Claude Code

Include automation/ in pytest coverage measurement alongside
core/ and api/ to track test coverage for workflow scripts
and generators.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings January 2, 2026 23:55
@MarkusNeusinger MarkusNeusinger merged commit 471167e into main Jan 2, 2026
9 checks passed
@MarkusNeusinger MarkusNeusinger deleted the ci/add-automation-coverage branch January 2, 2026 23:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the automation/ folder to pytest coverage measurement in the CI pipeline. Previously, only core/ and api/ were tracked for coverage. This change enables tracking of test coverage for automation components including plot generators and workflow utilities.

Key Change:

  • Added --cov=automation flag to the pytest command in the CI test workflow

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.

2 participants