[Proposed] Feature: all handlers should inject a Ilogger

audaciaconsulting / Audacia.CodeAnalysis

MIT License

1 stars 4 forks source link

[Proposed] Feature: all handlers should inject a Ilogger #36

Open Audacia-RhysSmith opened 3 months ago

Audacia-RhysSmith commented 3 months ago

Added CA2255 to library editorconfig
Added ACL1400, CA2254 and CA2264 to editorconfig

Action	Done?
Code compiles and unit tests all pass locally	✔
Debug/console log code removed	✔
Commented out code removed	✔
Unit tests added/updated	✔
README updated	✔
Considered: - Performance - Security - Logging	✔
Licenses of any new/upgraded dependencies checked, with no copyleft dependencies introduced	✔

Audacia-RhysSmith commented 3 months ago

@jackpercy-acl kindly pointed out that this does need to be split, I'll split the config changes into a separate PR if we decide this is something we want to add to the code analysis package :)