Skip to content

Silence the allowscalar warning in the test setup - #6088

Merged
giordano merged 1 commit into
mainfrom
mg/silence-allowscalar-warning
Sep 28, 2026
Merged

giordano merged 1 commit into
mainfrom
mg/silence-allowscalar-warning

Conversation

@giordano

Copy link
Copy Markdown
Member

Same spirit as #6087, to make tests less noisy

GPUArraysCore.allowscalar(true) warns that allowing scalar indexing globally is discouraged. The test setup is included by every test file, so with ParallelTestRunner the same two-line warning appeared once per file in the CI log. The global setting is intentional there, so discard the warning.

`GPUArraysCore.allowscalar(true)` warns that allowing scalar indexing globally
is discouraged.  The test setup is included by every test file, so with
ParallelTestRunner the same two-line warning appeared once per file in the CI
log.  The global setting is intentional there, so discard the warning.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YT3BwDvhiaJUeJXHKwCb7A
@codecov

codecov Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.25%. Comparing base (610c2b3) to head (a0698dc).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6088   +/-   ##
=======================================
  Coverage   75.25%   75.25%           
=======================================
  Files         434      434           
  Lines       27282    27282           
=======================================
  Hits        20532    20532           
  Misses       6750     6750           
Flag Coverage Δ
buildkite 69.13% <ø> (ø)
distributed_tripolar 20.73% <ø> (-0.02%) ⬇️
julia 69.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@giordano giordano added the testing 🧪 Tests get priority in case of emergency evacuation label Sep 27, 2026
@giordano
giordano merged commit bacf60f into main Sep 28, 2026
18 checks passed
@giordano
giordano deleted the mg/silence-allowscalar-warning branch September 28, 2026 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing 🧪 Tests get priority in case of emergency evacuation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants