Skip to content

Conversation

lunika
Copy link
Member

@lunika lunika commented Aug 28, 2025

Purpose

The parameters field in the MalwareDetection model does not use a
callable as default value, triggering a fields.E010 warning. We have to
use the dict callable instead.

Proposal

  • πŸ—ƒοΈ((malware_detection) use dict callable for MalwareDetection parameters
  • πŸ”–(patch) bump release to 0.0.13

@lunika lunika requested a review from qbey August 28, 2025 14:05
@lunika lunika self-assigned this Aug 28, 2025
@lunika lunika added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Aug 28, 2025
The parameters field in the MalwareDetection model does not use a
callable as default value, triggering a fields.E010 warning. We have to
use the `dict` callable instead.
lunika added a commit that referenced this pull request Aug 28, 2025
Fixed

- πŸ—ƒοΈ(malware_detection) use dict callable for MalwareDetection
  defaut parameters #26
@lunika lunika force-pushed the fix/json-default-callable branch from c19f23c to b2e43c3 Compare August 28, 2025 14:07
Copy link
Member

@qbey qbey left a comment

Choose a reason for hiding this comment

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

oups ^^'

lunika added 2 commits August 29, 2025 10:14
The command git watchanged is deprecated and is flag to be removed soon.
It can be easily replace with git log
Fixed

- πŸ—ƒοΈ(malware_detection) use dict callable for MalwareDetection
  defaut parameters #26
@lunika lunika force-pushed the fix/json-default-callable branch from b2e43c3 to 11baeba Compare August 29, 2025 08:14
@lunika lunika merged commit cc94487 into main Aug 29, 2025
11 checks passed
@lunika lunika deleted the fix/json-default-callable branch August 29, 2025 08:20
lunika added a commit that referenced this pull request Aug 29, 2025
Fixed

- πŸ—ƒοΈ(malware_detection) use dict callable for MalwareDetection
  defaut parameters #26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants