a8n-tools/rus
8
2
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Activity Actions

fix(tests): silence unused-variable lint in shorten closure #39

Closed
David wants to merge 1 commit from fix/unused-app-closure-RUS-4 into main
pull from: fix/unused-app-closure-RUS-4
merge into: a8n-tools:main
Conversation 0 Commits 1 Files changed 1 +1 -1
David commented 2026-06-12 17:53:36 +02:00
Owner
Copy link

The shorten test closure in src/handlers/urls.rs bound app without
using it inside the closure body, tripping the unused-variables lint and
failing the build under -D warnings. Prefix the binding with an underscore
(_app) since the closure references the outer token, not the passed app.

#RUS-4

The `shorten` test closure in `src/handlers/urls.rs` bound `app` without using it inside the closure body, tripping the `unused-variables` lint and failing the build under `-D warnings`. Prefix the binding with an underscore (`_app`) since the closure references the outer `token`, not the passed app. #RUS-4
fix(tests): silence unused-variable lint in shorten closure
All checks were successful
Create release / Create release from merged PR (pull_request) Has been skipped
Details
963f4898c3 
The `shorten` test closure in `src/handlers/urls.rs` bound `app` without
using it inside the closure body, tripping the `unused-variables` lint and
failing the build under `-D warnings`. Prefix the binding with an underscore
(`_app`) since the closure references the outer `token`, not the passed app.

#RUS-4
nrupard closed this pull request 2026-06-12 17:55:08 +02:00
All checks were successful
Create release / Create release from merged PR (pull_request) Has been skipped
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
a8n-tools/rus!39
No description provided.