Skip to content

Only check changed paths (infra)#1733

Merged
p-gentili merged 3 commits intomainfrom
only_check_changed_paths
Feb 18, 2025
Merged

Only check changed paths (infra)#1733
p-gentili merged 3 commits intomainfrom
only_check_changed_paths

Conversation

@Hook25
Copy link
Copy Markdown
Collaborator

@Hook25 Hook25 commented Feb 17, 2025

Description

This makes the pipeline slightly less wasteful by only re-testing directories that did actually change.

Resolved issues

N/A

Documentation

N/A

Tests

See this commit for the pipeline in action: https://github.com/canonical/checkbox/actions/runs/13372119135
See this commit for the pipeline not running (as it should): 27bc0df

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.26%. Comparing base (c52cf57) to head (534a920).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1733   +/-   ##
=======================================
  Coverage   49.26%   49.26%           
=======================================
  Files         373      373           
  Lines       40434    40434           
  Branches     6830     6830           
=======================================
  Hits        19921    19921           
  Misses      19785    19785           
  Partials      728      728           
Flag Coverage Δ
checkbox-ng 69.81% <ø> (ø)
provider-base 25.16% <ø> (ø)
provider-sru 97.97% <ø> (ø)

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

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

@Hook25 Hook25 force-pushed the only_check_changed_paths branch from 534a920 to 27bc0df Compare February 17, 2025 14:14
Copy link
Copy Markdown
Collaborator

@p-gentili p-gentili left a comment

Choose a reason for hiding this comment

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

+1

@p-gentili p-gentili merged commit 3a9f7ef into main Feb 18, 2025
8 checks passed
@p-gentili p-gentili deleted the only_check_changed_paths branch February 18, 2025 08:47
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