Skip to content

Extend libcore coverage for core::fmt, Part 2#2149

Merged
bors-ferrocene[bot] merged 8 commits intomainfrom
jh/libcore-coverage
Feb 14, 2026
Merged

Extend libcore coverage for core::fmt, Part 2#2149
bors-ferrocene[bot] merged 8 commits intomainfrom
jh/libcore-coverage

Conversation

@Urhengulas
Copy link
Copy Markdown
Member

@Urhengulas Urhengulas commented Feb 11, 2026

Cover functions of library/core that were added while adding core::fmt to the subset. See the commit messages for what tests for what functions have been added.

Test coverage diff

Executing ./x test library/core library/alloc --coverage=library --no-doc on both this branch and main (b683ce3) shows that this PR changes coverage (with "count annotated lines as tested") like following:

main this PR Diff
Lines 97.61% (25691/26321 lines) 97.92% (25773/26321 lines) +0.31%
Fully Tested 5909 5918 +9
Partially tested 22 13 -9
Fully untested 113 113 +0
Fully ignored 316 316 +0
Total 6357 6357 +0

Comment thread library/core/src/fmt/num.rs Outdated
Copy link
Copy Markdown
Member

@Hoverbear Hoverbear left a comment

Choose a reason for hiding this comment

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

bors merge

bors-ferrocene Bot added a commit that referenced this pull request Feb 13, 2026
2149: Extend libcore coverage for `core::fmt`, Part 2 r=Hoverbear a=Urhengulas

Cover functions of `library/core` that were added while adding `core::fmt` to the subset. See the commit messages for what tests for what functions have been added.

# Test coverage diff

Executing `./x test library/core library/alloc --coverage=library --no-doc` on both this branch and `main` (b683ce3) shows that this PR changes coverage (with "count annotated lines as tested") like following:

|                  | main                       | this PR                    | Diff  |
| ---------------- | -------------------------- | -------------------------- | ----- |
| Lines            | 97.61% (25691/26321 lines) | 97.92% (25773/26321 lines) | +0.31% |
| Fully Tested     | 5909                       | 5918                       | +9  |
| Partially tested | 22                        | 13                        | -9   |
| Fully untested   | 113                        | 113                        | +0   |
| Fully ignored    | 316                         | 316                         | +0    |
| Total            | 6357                       | 6357                       | +0  |

2152: Replace `print-target-features` with a run-make test r=tshepang a=jyn514

This test is highly specific to both the target and the exact version of LLVM installed. Rewrite it to only check that the feature boilerplate is printed, not the specific features shown.

As a bonus, the test is now target-independent and easy to run locally.

Co-authored-by: Urhengulas <[email protected]>
Co-authored-by: Jynn Nelson <[email protected]>
Co-authored-by: jyn <[email protected]>
@bors-ferrocene
Copy link
Copy Markdown
Contributor

Build failed (retrying...):

  • full

bors-ferrocene Bot added a commit that referenced this pull request Feb 14, 2026
2149: Extend libcore coverage for `core::fmt`, Part 2 r=Hoverbear a=Urhengulas

Cover functions of `library/core` that were added while adding `core::fmt` to the subset. See the commit messages for what tests for what functions have been added.

# Test coverage diff

Executing `./x test library/core library/alloc --coverage=library --no-doc` on both this branch and `main` (b683ce3) shows that this PR changes coverage (with "count annotated lines as tested") like following:

|                  | main                       | this PR                    | Diff  |
| ---------------- | -------------------------- | -------------------------- | ----- |
| Lines            | 97.61% (25691/26321 lines) | 97.92% (25773/26321 lines) | +0.31% |
| Fully Tested     | 5909                       | 5918                       | +9  |
| Partially tested | 22                        | 13                        | -9   |
| Fully untested   | 113                        | 113                        | +0   |
| Fully ignored    | 316                         | 316                         | +0    |
| Total            | 6357                       | 6357                       | +0  |

2155: Add s390x and ppc64le as supported targets r=tshepang a=Hoverbear

Add s390x and ppc64le as supported targets.

Co-authored-by: Urhengulas <[email protected]>
Co-authored-by: Ana Hobden <[email protected]>
@bors-ferrocene
Copy link
Copy Markdown
Contributor

Build failed (retrying...):

@bors-ferrocene
Copy link
Copy Markdown
Contributor

Build succeeded:

  • full

@bors-ferrocene bors-ferrocene Bot merged commit 648f1d0 into main Feb 14, 2026
3 of 4 checks passed
@bors-ferrocene bors-ferrocene Bot deleted the jh/libcore-coverage branch February 14, 2026 15:09
@github-actions github-actions Bot added backport:maybe PR candidate for backport merged-in:1.95 labels Feb 14, 2026
@Hoverbear Hoverbear removed the backport:maybe PR candidate for backport label Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants