Skip to content

Conversation

@DevelopmentCats
Copy link
Contributor

Description

Update CI workflow with automatic diff based detection for PR's so that only tests relating to affected files will be run in every PR greatly reducing the CI run time.

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

@DevelopmentCats
Copy link
Contributor Author

@matifali @Parkreiner

I think this should speed up our CI runs by a ton for a majority of PR's

@DevelopmentCats DevelopmentCats marked this pull request as ready for review October 14, 2025 18:39
@DevelopmentCats
Copy link
Contributor Author

The testing CI is a bug introduced by goose and will be fixed soon hopefully but everything else works.

block/goose#5169

@DevelopmentCats
Copy link
Contributor Author

The testing CI is a bug introduced by goose and will be fixed soon hopefully but everything else works.

block/goose#5169

this has been merged but their releases are hung up so we should be good soon for the failing CI.

Although this Workflow update should solve our issues on other PR's

Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

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

I would suggest using https://github.com/dorny/paths-filter instead.

@DevelopmentCats DevelopmentCats enabled auto-merge (squash) October 15, 2025 18:56
@matifali matifali requested a review from Copilot October 15, 2025 18:57
@matifali matifali disabled auto-merge October 15, 2025 18:58
Copy link
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 introduces automatic diff-based detection for CI workflows, enabling selective test execution based on changed files to reduce CI run times. The implementation adds a bash script that intelligently detects which TypeScript tests need to run and updates the GitHub Actions workflow to utilize this detection mechanism.

Key Changes:

  • Added selective test execution logic that runs only tests for changed modules
  • Integrated GitHub Actions paths-filter to identify affected files
  • Implemented fallback to full test suite when shared infrastructure changes

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
scripts/ts_test_auto.sh New bash script implementing intelligent test selection based on changed files with support for shared infrastructure detection
.github/workflows/ci.yaml Updated CI workflow to use paths-filter action and execute new test selection script with environment variables

@matifali matifali enabled auto-merge (squash) October 15, 2025 18:59
@matifali matifali merged commit 90873e8 into main Oct 15, 2025
4 checks passed
@matifali matifali deleted the cat/tf-test-ci-rework branch October 15, 2025 19:03
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