Merge pull request DataDog#4010 from jethrodaniel/clarify-yjit-ci-stats

clarify --yjit-stats=quiet in YJIT CI
p-datadog · Oct 18, 2024 · 167d8a5 · 167d8a5
2 parents 60febf8 + d403350
commit 167d8a5
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/.github/workflows/test-yjit.yaml b/.github/workflows/test-yjit.yaml
@@ -15,6 +15,10 @@ jobs:
           - '--yjit'
           - '--yjit --yjit-stats=quiet'
         exclude:
+          # NOTE: `--yjit-stats=quiet` is only present in 3.3+.
+          #   We've had `--yjit-stats` since 3.0, but
+          #   1) it dumps stat info to stdout, and
+          #   2) none of our <3.3 tests require the additional stats
           - ruby: '3.2'
             rubyopt: '--yjit --yjit-stats=quiet'
     name: Test YJIT (${{ matrix.os }}, ${{ matrix.ruby }} ${{ matrix.rubyopt }})