Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ADAP-1126: Filter pull request checks to only impacted packages #606

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mikealfare
Copy link
Contributor

Problem

Now that we have all of the adapters in the same repo, CI checks have grown in number. We should limit CI checks to only those that apply to affected packages. We can also use this as an opportunity to test changes to dependent packages, e.g. a change to dbt-adapters should trigger tests across all adapters.

Solution

  • Use the dorny/path-filters action that has been previously used to filter down to affected packages.
  • Apply different filters based on the check being done, e.g. dbt-tests-adapter has no unit tests nor integration tests, but does need a changelog

@mikealfare mikealfare self-assigned this Jan 14, 2025
@cla-bot cla-bot bot added the cla:yes The PR author has signed the CLA label Jan 14, 2025
@mikealfare mikealfare changed the title Filter pull request checks to only impacted packages ADAP-1126: Filter pull request checks to only impacted packages Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla:yes The PR author has signed the CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant