Skip to content

Conversation

@algebananazzzzz
Copy link

Fixes #13356

Outline of Solution

  1. Path-Based Triggering for Expensive Tests

The following workflows now have tighter path filters so they only run when application-impacting files change:

  • .github/workflows/axe.yml
  • .github/workflows/e2e.yml
  • .github/workflows/e2e-sql.yml

This helps reduce redundant runs caused by rebases, documentation-only changes, or unrelated updates, while preserving the same level of coverage and gating for our MRs .

# Only run when files relevant to E2E SQL tests are changed
paths:
# App code and templates that affect data flows
- 'src/main/java/**'
Copy link
Contributor

Choose a reason for hiding this comment

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

Our e2e-sql is basially a migration of our old e2e. Meaning to say e2e-sql has the same test coverage as e2e. I like your change, its forward thinking. Lets modify the coverage for e2e-sql and should be good on my end

Copy link
Contributor

Choose a reason for hiding this comment

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

@mingyuancode
Copy link
Contributor

Closing due to inactivity and branch being out of sync

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.

Optimize GitHub Actions

2 participants