Skip to content

Conversation

@papazof
Copy link
Contributor

@papazof papazof commented Jan 7, 2026

Remove custom session handling for resource cleanup. Since nubificus/vaccel#193 introduced automatic resource unregistration upon release, there is no longer need for the bugprone extra bookkeeping/cleanup for registered resource sessions.

@papazof papazof marked this pull request as ready for review January 7, 2026 18:33
@papazof papazof requested a review from ananos January 7, 2026 18:33
Copy link
Member

@ananos ananos left a comment

Choose a reason for hiding this comment

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

thanks @papazof

Remove custom session handling for resource cleanup. Since
nubificus/vaccel#193 introduced automatic resource unregistration upon
release, there is no longer need for the bugprone extra
bookkeeping/cleanup for registered resource sessions.

PR: #31
Signed-off-by: Kostis Papazafeiropoulos <papazof@gmail.com>
Reviewed-by: Anastassios Nanos <ananos@nubificus.co.uk>
Approved-by: Anastassios Nanos <ananos@nubificus.co.uk>
@github-actions github-actions bot force-pushed the refactor_remove_custom_resource_cleanup branch from f878d36 to fe3e51c Compare January 7, 2026 20:52
@vaccel-bot vaccel-bot bot added ok-to-test and removed ok-to-test labels Jan 7, 2026
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Coverage report diff

Filename Stmts Miss Cover
resource.py -11 -1 -1.31%
session.py 0 -1 +1.39%
TOTAL -11 -2 -0.05%

@ananos ananos merged commit 1c9f062 into main Jan 7, 2026
43 of 44 checks passed
@ananos ananos deleted the refactor_remove_custom_resource_cleanup branch January 7, 2026 21:50
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.

3 participants