andrew/libakerror
1
0
Fork 0
Code Issues 10 Pull Requests Actions Packages Projects Releases Wiki Activity

Test ignored-error storage across translation units #40

Merged
andrew merged 2 commits from 37 into main 2026-08-06 06:37:21 -04:00
Conversation 1 Commits 2 Files Changed 6 +51 -2
logikoma commented 2026-08-05 23:36:36 -04:00
Collaborator
Copy Link

Closes #37\n\nAdds a two-translation-unit regression test for the public ignored-error TLS slot and compiles it with -Wall -Wextra -Werror.

Closes #37\n\nAdds a two-translation-unit regression test for the public ignored-error TLS slot and compiles it with -Wall -Wextra -Werror.
logikoma added 3 commits 2026-08-05 23:36:36 -04:00
Add ASan and UBSan CI coverage
All checks were successful
libakerror CI Build / cmake_build (push) Successful in 2m52s
Details
libakerror CI Build / sanitizer (push) Successful in 2m56s
Details
libakerror CI Build / coverage (push) Successful in 2m57s
Details
libakerror CI Build / mutation_test (push) Successful in 37m44s
Details
07fdc82973
Pass CMake arguments to mutation builds
Some checks failed
libakerror CI Build / cmake_build (push) Successful in 2m57s
Details
libakerror CI Build / sanitizer (push) Successful in 3m4s
Details
libakerror CI Build / coverage (push) Successful in 2m49s
Details
libakerror CI Build / mutation_test (push) Has been cancelled
Details
576722ee06
Test ignored-error storage across translation units
Some checks failed
libakerror CI Build / cmake_build (push) Has been cancelled
Details
libakerror CI Build / sanitizer (push) Has been cancelled
Details
libakerror CI Build / coverage (push) Has been cancelled
Details
libakerror CI Build / mutation_test (push) Has been cancelled
Details
6dc22d5dbb
logikoma force-pushed 37 from 6dc22d5dbb to 360026508b 2026-08-05 23:36:53 -04:00 Compare
logikoma requested review from andrew 2026-08-05 23:37:00 -04:00
logikoma force-pushed 37 from 360026508b to bab2f102a1 2026-08-05 23:38:12 -04:00 Compare
andrew approved these changes 2026-08-06 06:37:13 -04:00
andrew left a comment
Owner
Copy Link

The test is kind of a useless hoop but sure it’ll catch a regression

The test is kind of a useless hoop but sure it’ll catch a regression
andrew merged commit 9249f8c029 into main 2026-08-06 06:37:21 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
andrew
Labels
Clear labels
abi-break
Cannot land without a soname bump
 api-gap
A caller cannot express something
 blast-radius:high
Ordering: highest blast radius
 blast-radius:low
Ordering: lowest blast radius
 blast-radius:medium
Ordering: middling blast radius
 defect
Wrong behaviour reachable from the public API
 design-decision
Needs a decision written down before code
 docs
Documentation, comments, AGENTS.md
 hygiene
Style, dead code, consistency
 packaging
Build, install, pkg-config, CMake config
 performance
Has a measurement behind it
 status::blocked
Waiting on something else -- a decision, a dependency, or another issue
 status::grooming
Not yet worked out: scope, approach or priority still being settled
 status::ready
Understood well enough to start: scope and approach are settled
 status::review
A pull request has been generated and this ticket is waiting on review of that PR
 test-coverage
A gap in the suites, not a defect
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
logikoma tachikoma andrew
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: andrew/libakerror#40
Delete Branch "37"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?