Skip to content

Add @nospecialize to conformance tests#2327

Merged
fingolfin merged 1 commit into
masterfrom
mh/nospec
Feb 4, 2026
Merged

Add @nospecialize to conformance tests#2327
fingolfin merged 1 commit into
masterfrom
mh/nospec

Conversation

@fingolfin
Copy link
Copy Markdown
Member

Reduces the overhead for the conformance tests in CI. These are big functions which Julia has to spend a lot of effort on for no benefit.

Reduces the overhead for the conformance tests in CI.
These are big functions which Julia has to spend a lot
of effort on for no benefit.
@fingolfin fingolfin added the release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes label Feb 3, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.16%. Comparing base (f890089) to head (0e06001).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2327   +/-   ##
=======================================
  Coverage   88.15%   88.16%           
=======================================
  Files         126      126           
  Lines       32050    32073   +23     
=======================================
+ Hits        28255    28277   +22     
- Misses       3795     3796    +1     

☔ View full report in Codecov by Sentry.
📢 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.

@fingolfin fingolfin merged commit 4d130f8 into master Feb 4, 2026
25 checks passed
@fingolfin fingolfin deleted the mh/nospec branch February 4, 2026 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants