Skip to content

feat: Add filtering for unchanged apps in unit tests - #109

Draft
amattu2 wants to merge 10 commits into
3.7.0from
feat/skip-unchanged-apps
Draft

feat: Add filtering for unchanged apps in unit tests#109
amattu2 wants to merge 10 commits into
3.7.0from
feat/skip-unchanged-apps

Conversation

@amattu2

@amattu2 amattu2 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Overview

This PR adjusts the unit test workflow to address some minor issues:

  1. If a app does not have any changes to itself or dependents, skip running unit tests for it
  2. Upgrade the outdated actions dependencies that rely on Node 20 (e.g. checkout)
  3. Remove unnecessary slack notification from workflow

TODO:

  • Test different changes to apps
  • Squash and recommit all of the testing commits into one

@amattu2
amattu2 force-pushed the feat/skip-unchanged-apps branch from adfc14b to 33e783f Compare August 6, 2026 14:07
@relativeci

relativeci Bot commented Aug 14, 2026

Copy link
Copy Markdown

#323 Bundle Size — 12.89MiB (0%).

a664fc0(current) vs ec7f608 3.7.0#322(baseline)

Warning

Bundle contains 54 duplicate packages – View duplicate packages

Bundle metrics  no changes
                 Current
#323
     Baseline
#322
No change  Initial JS 879.62KiB 879.62KiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 0% 28.67%
No change  Chunks 75 75
No change  Assets 118 118
No change  Modules 9174 9174
No change  Duplicate Modules 0 0
No change  Duplicate Code 0% 0%
No change  Packages 320 320
No change  Duplicate Packages 53 53
Bundle size by type  no changes
                 Current
#323
     Baseline
#322
No change  JS 10.37MiB 10.37MiB
No change  IMG 1.72MiB 1.72MiB
No change  Fonts 805.07KiB 805.07KiB
No change  CSS 18KiB 18KiB

Bundle analysis reportBranch feat/skip-unchanged-appsProject dashboard


Generated by RelativeCIDocumentationReport issue

This reverts commit a664fc0.
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.

1 participant