[PR #11] [MERGED] Fix: remove empty list as default #13

New issue

Closed

opened 2026-02-27 23:18:42 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 23:18:42 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/sophos/sophos-firewall-sdk/pull/11
Author: @mamullen13316
Created: 9/20/2023
Status: ✅ Merged
Merged: 9/20/2023
Merged by: @mamullen13316

Base: main ← Head: fix_update_service_acl

📝 Commits (2)

2e8f5d2 fix: pylint says don't use empty list as default
4b810ee fix: update version to 0.1.10

📊 Changes

2 files changed (+7 additions, -2 deletions)

View changed files

📝 pyproject.toml (+1 -1)
📝 sophosfirewall_python/firewallapi.py (+6 -1)

📄 Description

Pylint complaining about use of empty list as default argument on update_service_acl method.

_{🔄 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/sophos/sophos-firewall-sdk/pull/11 **Author:** [@mamullen13316](https://github.com/mamullen13316) **Created:** 9/20/2023 **Status:** ✅ Merged **Merged:** 9/20/2023 **Merged by:** [@mamullen13316](https://github.com/mamullen13316) **Base:** `main` ← **Head:** `fix_update_service_acl` --- ### 📝 Commits (2) - [`2e8f5d2`](https://github.com/sophos/sophos-firewall-sdk/commit/2e8f5d23829db46d6dd1d46e191a2dd68914b2d2) fix: pylint says don't use empty list as default - [`4b810ee`](https://github.com/sophos/sophos-firewall-sdk/commit/4b810eee9b254cc81e66fd4fac2a2e67d34cc354) fix: update version to 0.1.10 ### 📊 Changes **2 files changed** (+7 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `pyproject.toml` (+1 -1) 📝 `sophosfirewall_python/firewallapi.py` (+6 -1) </details> ### 📄 Description Pylint complaining about use of empty list as default argument on `update_service_acl` method. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>