andrew/libakstdlib
1
0
Fork 0
Code Issues 33 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Remove stale TODO section references #41

Open
logikoma wants to merge 1 commits from 27 into main
pull from: 27
merge into: andrew:main
Conversation 0 Commits 1 Files Changed 3 +5 -6
logikoma commented 2026-08-03 13:21:35 -04:00
Collaborator
Copy Link

Closes #27.

Most originally reported header citations had already disappeared through intervening edits. Remove the final stale section-number references from the conversion, linked-list, and memory test headers while retaining their descriptive rationale.

Tests:

  • verified no TODO.md or obsolete section-number citations remain under src or tests
  • cmake configure and build
  • ctest (20/20)
Closes #27. Most originally reported header citations had already disappeared through intervening edits. Remove the final stale section-number references from the conversion, linked-list, and memory test headers while retaining their descriptive rationale. Tests: - verified no TODO.md or obsolete section-number citations remain under src or tests - cmake configure and build - ctest (20/20)
logikoma added 1 commit 2026-08-03 13:21:36 -04:00
Remove stale TODO section references
Some checks failed
libakstdlib CI Build / cmake_build (push) Has been cancelled
Details
libakstdlib CI Build / sanitizers (push) Has been cancelled
Details
libakstdlib CI Build / coverage (push) Has been cancelled
Details
libakstdlib CI Build / mutation_test (push) Has been cancelled
Details
1b01aa3b42
logikoma force-pushed 27 from 1b01aa3b42 to a95ed6d13a 2026-08-03 13:46:43 -04:00 Compare
All checks were successful
libakstdlib CI Build / cmake_build (push) Successful in 2m55s
Required
Details
libakstdlib CI Build / sanitizers (push) Successful in 2m52s
Required
Details
libakstdlib CI Build / coverage (push) Successful in 2m45s
Required
Details
libakstdlib CI Build / mutation_test (push) Successful in 12m9s
Required
Details
This pull request doesn't have enough required approvals yet. 0 of 1 approvals granted from users or teams on the allowlist.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 27:27
git checkout 27
Sign in to join this conversation.
Reviewers
No Reviewers
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
 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
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: andrew/libakstdlib#41
Delete Branch "27"

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?