[PR #217] [MERGED] Added some initial log implementation #383

Closed
opened 2026-10-13 19:56:50 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/werwolfby/monitorrent/pull/217
Author: @DSilence
Created: 4/24/2017
Status: ✅ Merged
Merged: 4/24/2017
Merged by: @werwolfby

Base: develop ← Head: feature/logs


📝 Commits (6)

  • 558219a chore(logs): implemented some initial logging
  • 8b3d655 chore(logs): added some execute logs
  • 980b1d8 chore(tests): restored 100% coverage
  • 38542c7 chore(logs): postmerge fixes, added more tests
  • f478cc2 fix(test): python 2 error
  • 42a04d7 chore(logs): pull request comments changes

📊 Changes

18 files changed (+324 additions, -199 deletions)

View changed files

📝 monitorrent/engine.py (+10 -0)
📝 monitorrent/plugin_managers.py (+8 -0)
📝 monitorrent/plugins/clients/deluge.py (+20 -31)
📝 monitorrent/plugins/clients/qbittorrent.py (+36 -47)
📝 monitorrent/plugins/clients/transmission.py (+18 -34)
📝 monitorrent/plugins/clients/utorrent.py (+17 -26)
📝 monitorrent/rest/clients.py (+18 -0)
📝 monitorrent/rest/execute.py (+42 -30)
📝 monitorrent/rest/notifiers.py (+14 -0)
📝 requirements.txt (+1 -0)
📝 server.py (+36 -7)
📝 tests/plugins/clients/test_deluge_plugin.py (+13 -6)
📝 tests/plugins/clients/test_qbittorrent.py (+9 -5)
📝 tests/plugins/clients/test_transmission_plugin.py (+15 -7)
📝 tests/plugins/clients/test_utorrent_plugin.py (+8 -5)
📝 tests/rest/test_api_clients.py (+34 -1)
📝 tests/rest/test_api_execute.py (+13 -0)
📝 tests/rest/test_api_notifiers.py (+12 -0)

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/werwolfby/monitorrent/pull/217 **Author:** [@DSilence](https://github.com/DSilence) **Created:** 4/24/2017 **Status:** ✅ Merged **Merged:** 4/24/2017 **Merged by:** [@werwolfby](https://github.com/werwolfby) **Base:** `develop` ← **Head:** `feature/logs` --- ### 📝 Commits (6) - [`558219a`](https://github.com/werwolfby/monitorrent/commit/558219a6d23f747f61ca53d4d08e395292e2763d) chore(logs): implemented some initial logging - [`8b3d655`](https://github.com/werwolfby/monitorrent/commit/8b3d655a05d6bb5c5818913e8fe380793f914022) chore(logs): added some execute logs - [`980b1d8`](https://github.com/werwolfby/monitorrent/commit/980b1d8c44ae5b1a1b907caece18b17cf53e2000) chore(tests): restored 100% coverage - [`38542c7`](https://github.com/werwolfby/monitorrent/commit/38542c7f8e73572ae57c12d2ea5692a99dc6603e) chore(logs): postmerge fixes, added more tests - [`f478cc2`](https://github.com/werwolfby/monitorrent/commit/f478cc2f122156ffcfff8e22f50ecace5ad0880f) fix(test): python 2 error - [`42a04d7`](https://github.com/werwolfby/monitorrent/commit/42a04d7014e134c9d50f265ec3055f7e2412b3c9) chore(logs): pull request comments changes ### 📊 Changes **18 files changed** (+324 additions, -199 deletions) <details> <summary>View changed files</summary> 📝 `monitorrent/engine.py` (+10 -0) 📝 `monitorrent/plugin_managers.py` (+8 -0) 📝 `monitorrent/plugins/clients/deluge.py` (+20 -31) 📝 `monitorrent/plugins/clients/qbittorrent.py` (+36 -47) 📝 `monitorrent/plugins/clients/transmission.py` (+18 -34) 📝 `monitorrent/plugins/clients/utorrent.py` (+17 -26) 📝 `monitorrent/rest/clients.py` (+18 -0) 📝 `monitorrent/rest/execute.py` (+42 -30) 📝 `monitorrent/rest/notifiers.py` (+14 -0) 📝 `requirements.txt` (+1 -0) 📝 `server.py` (+36 -7) 📝 `tests/plugins/clients/test_deluge_plugin.py` (+13 -6) 📝 `tests/plugins/clients/test_qbittorrent.py` (+9 -5) 📝 `tests/plugins/clients/test_transmission_plugin.py` (+15 -7) 📝 `tests/plugins/clients/test_utorrent_plugin.py` (+8 -5) 📝 `tests/rest/test_api_clients.py` (+34 -1) 📝 `tests/rest/test_api_execute.py` (+13 -0) 📝 `tests/rest/test_api_notifiers.py` (+12 -0) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-13 19:56:50 +00:00
Sign in to join this conversation.
No milestone
No project
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
werwolfby/monitorrent#383
No description provided.