dmwm / dmwm/FileMover

Clean-up ActivityMonitor/Monitor

Open
#804 0 comments 0 reactions 1 assignee Claimed by @vkuznet View on GitHub
core development
Dominant language
Python
Stars
4
Forks
0
PR merge metrics
No merged PRs in 30d

Description

From Lassi code audit notes

Concerning "So I'll post-pone this until you provide a proof that other methods are not in use", just grep the sources for "ActivityMonitor?". It's only imported in two places, one which makes no use of it, and the second of which makes use of unique_token() in Monitor. There are more uses of ActivityObject?, but it makes little useful use of Monitor.
I am tempted to rephrase the review comment to "please provide exact proof of which all methods are actually called by the server, and delete the rest" :-)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.